Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I've said earlier, I'd much rather be pulling this from an FSF API. In the absence of such an API (the current situation) I think we should put this information in a mock FSF API which we can hand over to the FSF when they're ready to step up and maintain it. I really don't like baking other people's metadata into our root data repository. I would much rather this information be included in license-list-data after being pulled from the (mock) FSF API, instead of including the metadata here.
I am ok baking in an FSF license ID (if they decide to diverge from SPDX IDs in their API), at least until we address #418.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree in principle, but I don't want to screen scrape for the information. We could have a mock API that would work from a config file. It should present the data through a web API, preferably REST based
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was imagining we dump some JSON into GitHub. I'll sketch that out tonight.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JSON works. If you put together sending side - I can update the SPDX license generator to use it.
@kestewart - Can you just put together a list of the licenses you think are FSF Libre?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll pull the list using JavaScript, as I showed on the mailing list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goneall - sure. Just a flat text file of one SPDX-license-identifier per line for the subset that are isFsfLibre?