Skip to content
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

llvm: 4.0rc2 -> 4rc3 #23408

Merged
merged 2 commits into from Mar 3, 2017
Merged

llvm: 4.0rc2 -> 4rc3 #23408

merged 2 commits into from Mar 3, 2017

Conversation

dtzWill
Copy link
Member

@dtzWill dtzWill commented Mar 2, 2017

Motivation for this change

LLVM published 4.0rc3 today, this updates us from rc2.
Also incorporates an idea from the rc2 PR and shortens the version to reflect the new versioning scheme used.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Also add myself to maintainers for LLVM.
This reflects upstream versioning change, and allows
us to replace 4.0 with 4.1 (which is now a minor revision)
without changing the attribute name.

Thanks to @vcunat for the idea.
vcunat added a commit to vcunat/nixpkgs that referenced this pull request Mar 3, 2017
and rename attributes 4.0 -> 4
@vcunat vcunat merged commit 6f6c06a into NixOS:master Mar 3, 2017
vcunat added a commit that referenced this pull request Mar 3, 2017
and rename attributes 4.0 -> 4

(cherry picked from commit 92f454e)
@andrewrk
Copy link
Member

andrewrk commented Mar 7, 2017

Thank you for being on top of this! This is one reason I love NixOS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants