-
Notifications
You must be signed in to change notification settings - Fork 109
No license detected in a docker scans of docker://redis:buster #478
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
Comments
Let's take a data example of an
|
Thanks for the inputs For 2., resources.package_data are no longer used correctly but the data should still be processed correctly. I will submit a fix An 3. is another SCTK bug |
* The license of Alpine installed system packages was not detected correctly. It now uses the Alpine specific procedure. Reference: aboutcode-org/scancode.io#478 Signed-off-by: Philippe Ombredanne <[email protected]>
* The license of Debian installed system packages was not detected correctly. In particular merging copyright detection details with other metadata was not working Reference: aboutcode-org/scancode.io#478 Signed-off-by: Philippe Ombredanne <[email protected]>
See aboutcode-org/scancode-toolkit#3036 for the fixes on the SCTK side |
Validate that things run OK and license are detected with a minimal apine and debian docker image. Also adapt existing tests to new results. Bump SCTK to latest version. Reference: #478 Signed-off-by: Philippe Ombredanne <[email protected]>
* Validate that things run OK and license are detected with a minimal alpine and debian docker image. * Also adapt existing tests to new results. * Bump SCTK to latest version 31 Reference: #478 Signed-off-by: Philippe Ombredanne <[email protected]>
Some of these changes may not be entirely correct and will need further review Reference: #478 Signed-off-by: Philippe Ombredanne <[email protected]>
This minimizes changes in diffs Signed-off-by: Philippe Ombredanne <[email protected]>
* Add tests for system package licenses #478 * Validate that things run OK and license are detected with a minimal alpine and debian docker image. * Also adapt existing tests to new results. * Bump SCTK to latest version 31 Reference: #478 Signed-off-by: Philippe Ombredanne <[email protected]> * Sort expected test results #478 This minimizes changes in diffs Signed-off-by: Philippe Ombredanne <[email protected]> * Bump ScanCode toolkit to 31.0.0rc5 * also adjust tests accordingly Signed-off-by: Philippe Ombredanne <[email protected]>
I created a project and scanned
docker://redis:buster
with a docker pipeline.No license was detected at all using the tip of the main branch
The text was updated successfully, but these errors were encountered: