-
Notifications
You must be signed in to change notification settings - Fork 187
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
Consider switching back to MIT license #358
Comments
The de facto standard in rustland is to dual license under Apache 2 and MIT, Apache preferred for patent troll protections, the latter for GPLv2 compatibility. Seems like we should do the same, unless there's a compelling reason not to.
|
Reopening this to discuss GPL3. |
Meh..that would be like the opposite of MIT haha. Do you think we are currently in breach of the Solidity license? I guess we are because If that's the case then I suggest we raise the issue internally in the EF and find ways to mitigate it. My naive thinking would be that Solidity could grant us exceptional rights or basically just say "we won't go against you" but what I don't know is if that would prevent an angry external observer (someone totally unrelated to the Solidity team) to still turn the license issue against us. But I assume the EF legal team can help us figure that out. |
@g-r-a-n-t and I discussed this a bit on discord. Everything that depends on yultsur/solc-rust/solc (directly or transitively) has to be licensed GPL3. I suggest we split the yul/solc codegen stuff out into a separate crate, and add a driver crate that pulls everything together (the current
We can't use solidity under a different license unless every contributor agrees to relicense their contributions. The Fe project can't claim that code is usable under the terms of apache2/mit if it's actually bound by the terms of gpl3. |
@sbillig cool, that makes sense! I like it 👍 |
There are only three contributors to our fork of yultsur. We might be able to get everyone to sign off on a switch to apache 2 and then structure things like so:
|
What is wrong?
I believe the only reason why we migrated away from the MIT license was because the parser incorporated code from a project (nom) that isn't MIT licensed. With the parser being rewritten we could consider switching back.
How can it be fixed
Discuss & change.
The text was updated successfully, but these errors were encountered: