Skip to content

Point to a list of the actual license identifiers #31

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

Merged
merged 1 commit into from
Nov 14, 2014

Conversation

huonw
Copy link
Member

@huonw huonw commented Nov 14, 2014

The format cargo understands is not necessarily obvious from the license name so it is good to point to a place listing the exact text.

The format cargo understands is not necessarily obvious from the license name so it is good to point to a place listing the exact text.
@alexcrichton
Copy link
Member

I originally wanted to avoid recommending spdx as I'm not certain of how "official" it is in terms of being accepted among the open source community as a standard for license names. All of the short identifiers listed on opensource.org are accepted as well.

I also wanted to suggest the official opensource.org recommended licenses (/licenses) over all the licenses (/licenses/alphabetical). This all may end up being somewhat confusing though... Perhaps a suggestion like:

unknown license `{}`, see http://opensource.org/licenses for options, and http://spdx.org/licenses/ for some recommended identifiers

I'm not sure if mentioning spdx is overkill though...

@alexcrichton
Copy link
Member

I think I may just be overthinking this, let's just take it. Thanks @huonw!

alexcrichton added a commit that referenced this pull request Nov 14, 2014
Point to a list of the actual license identifiers
@alexcrichton alexcrichton merged commit 302d9de into rust-lang:master Nov 14, 2014
Turbo87 added a commit that referenced this pull request Dec 8, 2022
We can just rely on the `extensions` field in the `Parts` struct
Turbo87 pushed a commit that referenced this pull request Jan 4, 2023
…-0.9.x

Update Rust crate conduit-router to 0.9.0-alpha.6
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