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

Failed to import SPDX "PACKAGE-MANAGER" vs. "PACKAGE_MANAGER" #1130

Closed
pombredanne opened this issue Mar 26, 2024 · 1 comment
Closed

Failed to import SPDX "PACKAGE-MANAGER" vs. "PACKAGE_MANAGER" #1130

pombredanne opened this issue Mar 26, 2024 · 1 comment

Comments

@pombredanne
Copy link
Member

We cannot import some SPDX docs because of a small mess between 2.3 and 2.3.1 schemas in JSON and other parts.
We also need to decide on the best format for output.

We should accept PACKAGE-MANAGER and PACKAGE_MANAGER on export.

See the mini mess here:

Basically thanks to this, both should be supported on reading.

I am not sure which one is the best to export/write as a standard.

In all cases we would need to support both 2.3 and 2.3.1 import and decide on the most popular output string to use.

tdruez added a commit that referenced this issue Mar 28, 2024
Signed-off-by: tdruez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Mar 28, 2024
Signed-off-by: tdruez <tdruez@nexb.com>
tdruez added a commit that referenced this issue Mar 28, 2024
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Copy link
Contributor

tdruez commented Mar 28, 2024

Schema updated in #1139
Both "PACKAGE-MANAGER" and "PACKAGE_MANAGER" are now valid.

@tdruez tdruez closed this as completed Mar 28, 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