Skip to content

Zephyr SPDX does not pass validation #89413

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

Open
JarmouniA opened this issue May 2, 2025 · 0 comments
Open

Zephyr SPDX does not pass validation #89413

JarmouniA opened this issue May 2, 2025 · 0 comments
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug Licensing The PR has licensing issues => licensing expert to review priority: medium Medium impact/importance bug

Comments

@JarmouniA
Copy link
Collaborator

Discussed in #89371

Originally posted by im-sampm May 1, 2025
Hello! I wanted to bring it to the team's attention that the current Zephyr SPDX file does not pass validation.

To recreate this bug, download the zephyr-v4.1.0.spdx file here: link

And upload it to the SPDX validator here: link (using the File Type "V2 Tag/Value").

Which returns the following error:
image

My team was able to narrow the issue down to this line in the SPDX:
image

This line is generated from the following file:

* SPDX-License-Identifier: Apache-2.0 and UNLICENSED

According to the SPDX v2 Spec, UNLICENSED is an invalid entry because it is not part of the SPDX License List and if it is a custom license, it should be prefixed with LicenseRef-.

This is also the only file which includes the UNLICENSED entry: link

To fix this issue, the license should either be removed or updated IN qemu_kvm_arm64.overlay to match the SPDX spec. Until then, every SPDX file generated by the Zephyr project will be out of spec.

Here is the git blame for the line:

image

@mikeg-infomagnus @sivakinfomagnus

@henrikbrixandersen henrikbrixandersen added bug The issue is a bug, or the PR is fixing a bug Licensing The PR has licensing issues => licensing expert to review labels May 2, 2025
@dkalowsk dkalowsk added the priority: medium Medium impact/importance bug label May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug Licensing The PR has licensing issues => licensing expert to review priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

3 participants