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

pull: Add deprecated SPDX identifiers #14

Merged
merged 1 commit into from
Apr 15, 2018
Merged

pull: Add deprecated SPDX identifiers #14

merged 1 commit into from
Apr 15, 2018

Conversation

wking
Copy link
Owner

@wking wking commented Apr 13, 2018

These identifiers are deprecated, but they haven't yet been dropped from the SPDX spec. That means that, although they're not recommended, they're still valid identifiers to use where you can use other License List 3.0 identifiers. Listing them here makes it easy for folks using the deprecated identifiers to look up the associated FSF metadata.

An alternative approach would be to stick with only non-deprecated SPDX identifiers here and have users hit the SPDX API to resolve those before coming to this API. But the SPDX API does not currently expose obsoletedBy migration recommendations (spdx/LicenseListPublisher#12).

CC @goneall.

These identifiers are deprecated, but they haven't yet been dropped
from the SPDX spec.  That means that, although they're not
recommended, they're still valid identifiers to use where you can use
other License List 3.0 identifiers.  Listing them here makes it easy
for folks using the deprecated identifiers to look up the associated
FSF metadata.

An alternative approach would be to stick with only non-deprecated
SPDX identifiers here and have users hit the SPDX API to resolve those
before coming to this API.  But the SPDX API does not currently expose
obsoletedBy migration recommendations [1].

[1]: spdx/LicenseListPublisher#12
@goneall
Copy link
Contributor

goneall commented Apr 15, 2018

Agree with adding the deprecated ID's as a solution. We can pick these up on the next update of the license list data.

@wking wking merged commit 46edc2d into master Apr 15, 2018
@wking wking deleted the deprecated-spdx-ids branch April 15, 2018 04:53
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

Successfully merging this pull request may close these issues.

2 participants