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

LicenseListVersion: 3.6 #3257

Closed
vargenau opened this issue Feb 15, 2023 · 3 comments · Fixed by #3271
Closed

LicenseListVersion: 3.6 #3257

vargenau opened this issue Feb 15, 2023 · 3 comments · Fixed by #3271
Labels

Comments

@vargenau
Copy link
Contributor

Description

In SPDX, we have:

LicenseListVersion: 3.6

That should be:

LicenseListVersion: 3.19

How To Reproduce

svn checkout https://svn.code.sf.net/p/phpwiki/code/trunk phpwiki
./scancode -c -l -i --license-text --spdx-tv phpwiki.spdx phpwiki

Resulting SPDX file:

phpwiki.spdx.txt

System configuration

./scancode --version
ScanCode version: 32.0.0rc1
ScanCode Output Format version: 3.0.0
SPDX License list version: 3.19

Ubuntu 22.10

@vargenau vargenau added the bug label Feb 15, 2023
AyanSinhaMahapatra added a commit that referenced this issue Feb 22, 2023
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra
Copy link
Member

This bug was caused because SPDX python was using the license list version from it's own config and silently ignoring the SPDX license list version passed by scancode-toolkit. This should be fixed by #3271 where we are now forcing this value onto Document.CreationInfo()

@vargenau
Copy link
Contributor Author

Thank you for the fix and the explanation. It was very strange to have such an old license list version.

I have opened a bug: spdx/tools-python#491

@vargenau
Copy link
Contributor Author

I have fixed it in tools-python: spdx/tools-python#492

It will be in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants