-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
llvmPackages: add asl20-llvm
to licenses for v19+
#314347
base: master
Are you sure you want to change the base?
Conversation
As per llvm/llvm-project#92394 contributions made after June 1st, 2024 are licensed exclusively under `Apache-2.0 WITH LLVM-exception` rather than dual licensed. As per this post, this change is timed so that it only affects version 19 and newer: https://discourse.llvm.org/t/relicensing-next-step-dropping-requirement-to-contribute-also-under-the-legacy-license/78351
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.
Sounds good though wouldn't a better fit be with an LLVM git update PR rather than it's own PR?
I didn't want us to forget to make this change and I wanted another pair of eyes on this change specifically — I've made mistakes here before. |
Sounds good, LGTM. Meanwhile, I'll try to fix the latest LLVM git update so we can merge that ASAP. Hopefully that'll make LLVM 19 smooth sailing when that releases in like a month or two. |
@alyssais: friendly ping |
Description of changes
As per llvm/llvm-project#92394 contributions made after June 1st, 2024 are licensed exclusively under
Apache-2.0 WITH LLVM-exception
rather than dual licensed.As per this post, this change is timed so that it only affects version 19 and newer.
If I'm understanding correctly, I think this means we should add
licenses.asl20-llvm
tollvm_meta.license
forrelease_version >= 19
.Previously: #217906.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.