-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Copyright sign in GPL-3.0 does not match all instances #260
Comments
It does, by means of the Matching Guidelines (https://spdx.github.io/spdx-spec/appendix-II-license-matching-guidelines-and-templates/#9-copyright-symbol) which explicitly say:
|
What about an uppercase C, as in |
This is covered by Matching Guidelines section 4:
|
Note that many of the guidelines are not implemented in the templates themselves, but are implemented in tools which support the license matching guidelines. There are now 3 tools that compare license text using the matching guidelines and text:
|
I'm using the Java library to detect licenses with I have a unit test that tries to detect this license text, which starts with The test fails with the license template obtained via spdx-tools. I made a slight change in line 5 to make the test pass: So it seems to me that the matching guidelines are not fully implemented in this case. |
@spoenemann Thanks for the detailed description and unit test reference. The Java matching tools should be normalizing the copyright and case. I'll look into the code over the weekend and see if I can find out why it isn't matching. |
I found the issue in the SPDX java tools - transferring to the tools repo |
…ense matcher Signed-off-by: Gary O'Neall <[email protected]>
@spoenemann Finally got this resolved in PR #261 - if you pull the latest code, it should now work. Let me know if you want me to spin out a new release to Maven Central and Bintray |
Great! It would be good to have this in a release at some time, but it's not urgent as we have a workaround. |
Resolved in release 2.2.4 |
The current license template of GPL-3.0 contains the line
However, the copyright sign
©
is represented as(C)
in many instances of the license found in the wild. The template should be able to match both variants.The text was updated successfully, but these errors were encountered: