-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
c3c: 0.5.5 -> 0.6.1 #330456
c3c: 0.5.5 -> 0.6.1 #330456
Conversation
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/4326 |
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package built:
|
This makes sure the hook will catch cases where the --version or --help output prints the installation dir or something alike that includes the builtins.storeDir, and later on the `${pname}-${version}` string. Could have avoided NixOS#334245 which was a fixup for NixOS#330456.
Description of changes
Update the C3 compiler to the latest version and reformat the package with
nixfmt-rfc-style
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.
CC: @luc65r