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

Use SPDX JSON license list #1

Closed
flukejones opened this issue Sep 3, 2019 · 2 comments · Fixed by #4
Closed

Use SPDX JSON license list #1

flukejones opened this issue Sep 3, 2019 · 2 comments · Fixed by #4
Assignees
Labels
enhancement New feature or request

Comments

@flukejones
Copy link

Is your feature request related to a problem? Please describe.
I was considering using cargo-deny to help with packaging crates for a distro, which while incredibly helpful, would be made even more so by including the isOsiApproved field available in the SPDX data.

Where this would be helpful is I could make a service on OBS, and set a series of restrictions such as "must be isOsiApproved and in the approved list" or similar. It would help to select a broad range of licenses quickly.

Describe the solution you'd like
Either add this as an additional field in the existing license data, or parse the SPDX (perhaps pub const DATA = include!("licenses.json"); ?). I guess you'd need to weigh up the pros/cons of including a large text file in the binary vs just the required data. Or perhaps use a build.rsto parse it, and spit out anidentifiers.rs` with only the required data.

@flukejones flukejones added the enhancement New feature or request label Sep 3, 2019
@Jake-Shadle
Copy link
Member

Agreed, I was going to put that in cargo-deny originally but never did. The current identifiers file is actually copied from the license exprs crate, so yah, would need to generate it in this crate itself to get more info.

@flukejones
Copy link
Author

Thanks so much for considering this.

This was referenced Sep 17, 2019
Jake-Shadle added a commit that referenced this issue Oct 3, 2019
* Use a better lexer, taken from ehuss/license-exprs#29
* Add a simple parser/evaluator
* Add an update exe for pulling new SDPX information, also taken from https://github.com/rust-lang-nursery/license-exprs
* Add support for some of the metadata available from the SPDX format, namely "IsDeprecated", "IsFSFLibre", "IsOSIApproved"

Resolves: #1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants