camallo / dkregistry-rs

A pure-Rust asynchronous library for Docker Registry API v2
Apache License 2.0
62 stars 39 forks source link

docs: clarify licensing #164

Closed LalatenduMohanty closed 4 years ago

LalatenduMohanty commented 4 years ago

This adds license files and information to the README.

Closes: #143

lucab commented 4 years ago

@LalatenduMohanty thank for the cleanups. However I think #143 is just asking us to make the license more prominent, not to relicense the whole crate (MIT OR APL2 is fine, and better aligned with the larger Rust ecosystem).

I think you can follow the approach from other repos (this is a personal example I have) and mirror it here, like this:

LalatenduMohanty commented 4 years ago

Thanks @lucab . It is interesting that Rust ecosystem prefers dual licensing.

lucab commented 4 years ago

@LalatenduMohanty that is documented in the language guidelines. It shouldn't really matter for Cincinnati, as the consumer can pick which of the two it prefers.

LalatenduMohanty commented 4 years ago

@lucab Thanks, I have updated the PR as per https://rust-lang.github.io/api-guidelines/necessities.html#crate-and-its-dependencies-have-a-permissive-license-c-permissive