-
Notifications
You must be signed in to change notification settings - Fork 599
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
SPDX PackageLicenseDeclared should be NOASSERTION #660
Comments
Thanks @rnjudge for the find here! We've got some work in-flight regarding identifying and propagating license information info the SPDX format/base syft data shape. I think we can do a quick patch that gets I can get this in when I have some bandwidth this week. Also, if it seems small enough and you're interested in becoming a contributor, feel free to throw a PR up. I'd be happy to review it and help merge it in for our next release. |
@spiffcs sounds good! I can do that before vacation next week :) |
What happened:
Running
syft photon:3.0 -o spdx
thePackageLicenseDeclared
for all packages is listed asNONE
. According to the SPDX spec, however,NONE
should only be used if "the package contains no license information whatsoever". The photon packages do contain license information, however, soNOASSERTION
should be the value ofPackageLicenseDeclared
since a license is available but not provided.What you expected to happen:
At a minimum,
NOASSERTION
should be the value of PackageLicenseDeclared (and probably PackageLicenseConcluded). Ideally, the License or LicenseRef for the package would be listed since the license is provided in the json format:How to reproduce it (as minimally and precisely as possible):
syft photon:3.0 -o spdx
Anything else we need to know?:
https://spdx.github.io/spdx-spec/package-information/
Version: 0.32.0
The text was updated successfully, but these errors were encountered: