-
Notifications
You must be signed in to change notification settings - Fork 40
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
switch licenses to mit-apl #13
Conversation
see related rs-ipfs/rust-ipfs#50
You were faster then I was, I also had the PR already prepared but haven't pushed it yet. For those wondering why, I'd like to quote from ipfs/js-ipfs#2624, for the reasons for dual-licensing under MIT/Apache2:
|
Best would be if @dvc94ch could approve this PR and perhaps even comment:
|
We discussed on riot, I'm not sure how dual licensing provides patent protection, since I can use rust-ipfs under the MIT license and still sue rust-ipfs for patent infringement. But as I said, it's ok with me |
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.
@koivunej Could you please also add the license = "MIT OR Apache-2.0"
to all crates in this repo?
All Cargo.tomls now have the license, failed with my sed magic the first time but now they seem to be ok :) |
see related rs-ipfs/rust-ipfs#50.
There is only a single contributor before this so probably ok just to merge.