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

license-checker-tool: any x11 license that does not specifically say "X Consortium" will be detected as x11-distribute-modifications #540

Open
quantenzitrone opened this issue Oct 27, 2024 · 2 comments

Comments

@quantenzitrone
Copy link

try for example https://gitlab.freedesktop.org/xorg/lib/pthread-stubs/-/blob/master/COPYING

even though it doesn't have the defining change for x11-distribute-modifications:

+ [...] distribute, distribute with modifications, sublicense [...]
- [...] distribute, sublicense [...]

it is still detected as x11-distribute-modifications

goneall added a commit to spdx/Spdx-Java-Library that referenced this issue Oct 27, 2024
Added to see if the library is the cause of
spdx/spdx-online-tools#540

Signed-off-by: Gary O'Neall <[email protected]>
@goneall
Copy link
Member

goneall commented Oct 27, 2024

Thanks @quantenzitrone for reporting the issue.

I verified the online tools does indeed match the x11-distribute-modifications when it should not.

The online tools should be calling the org.spdx.utility.compare.LicenseCompareHelperTest method in the Java library.

I wrote a Unit Test and verified that the Java library reports this as not matching.

Note that I added a unit test and tested it against a much later version of the SPDX Java Library. It passes the unit test, so either the new library will fix this, or there is an issue in the Python code which should be calling the SPDX Java Library.

@goneall
Copy link
Member

goneall commented Oct 27, 2024

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

2 participants