Skip to content
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

Closed
spoenemann opened this issue Oct 7, 2020 · 10 comments
Closed

Copyright sign in GPL-3.0 does not match all instances #260

spoenemann opened this issue Oct 7, 2020 · 10 comments

Comments

@spoenemann
Copy link

The current license template of GPL-3.0 contains the line

Copyright © 2007 Free Software Foundation, Inc.

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.

@zvr
Copy link
Member

zvr commented Oct 7, 2020

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:

"©", "(c)", or "Copyright" should be considered equivalent and interchangeable.

@spoenemann
Copy link
Author

What about an uppercase C, as in (C)?

@goneall
Copy link
Member

goneall commented Oct 7, 2020

What about an uppercase C, as in (C)?

This is covered by Matching Guidelines section 4:

All upper case and lower case letters should be treated as lower case letters. Templates do not include markup for this guideline.

@goneall
Copy link
Member

goneall commented Oct 7, 2020

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:

@spoenemann
Copy link
Author

I'm using the Java library to detect licenses with LicenseCompareHelper here:

https://github.com/eclipse/openvsx/blob/37288e214643c943423b29c55258899e36375460/server/src/main/java/org/eclipse/openvsx/util/LicenseDetection.java#L65

I have a unit test that tries to detect this license text, which starts with Copyright (C):

https://github.com/eclipse/openvsx/blob/master/server/src/test/resources/org/eclipse/openvsx/util/GPL-3.0.txt

The test fails with the license template obtained via spdx-tools. I made a slight change in line 5 to make the test pass:

https://github.com/eclipse/openvsx/blob/master/server/src/main/resources/spdx-templates/GPL-3.0.txt

So it seems to me that the matching guidelines are not fully implemented in this case.

@goneall
Copy link
Member

goneall commented Oct 8, 2020

@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.

@goneall
Copy link
Member

goneall commented Oct 25, 2020

I found the issue in the SPDX java tools - transferring to the tools repo

@goneall goneall transferred this issue from spdx/license-list-XML Oct 25, 2020
goneall added a commit that referenced this issue Oct 25, 2020
@goneall
Copy link
Member

goneall commented Oct 25, 2020

@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

@spoenemann
Copy link
Author

Great! It would be good to have this in a release at some time, but it's not urgent as we have a workaround.

@goneall
Copy link
Member

goneall commented Nov 14, 2020

Resolved in release 2.2.4

@goneall goneall closed this as completed Nov 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants