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

setup-hooks/reproducible-builds.sh: NIX_OUTPATH_USED_AS_RANDOM_SEED #216016

Merged
merged 1 commit into from Feb 18, 2023
Merged

setup-hooks/reproducible-builds.sh: NIX_OUTPATH_USED_AS_RANDOM_SEED #216016

merged 1 commit into from Feb 18, 2023

Conversation

ghost
Copy link

@ghost ghost commented Feb 12, 2023

Description of changes

For reproducibility, nixpkgs sets -frandom-seed to the first 10 characters of the outpath of the derivation being built.

This PR allows to manually select from which outpath the 10-character prefix is taken, by setting NIX_OUTPATH_USED_AS_RANDOM_SEED as a derivation attribute (or directly as an environment variable).

For an example use of this functionality, see #209870

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

For reproducibility, nixpkgs sets `-frandom-seed` to the first 10
characters of the outpath of the derivation being built.

This PR allows to manually select from which outpath the
10-character prefix is taken, by setting
`NIX_OUTPATH_USED_AS_RANDOM_SEED` as a derivation attribute (or
directly as an environment variable).

For an example use of this functionality, see:

  #209870
@trofi trofi merged commit c8ca871 into NixOS:master Feb 18, 2023
@yu-re-ka
Copy link
Contributor

This was accidentally merged into master?

@trofi
Copy link
Contributor

trofi commented Feb 18, 2023

Gah, you are right! Thank you for a prompt revert!

@trofi
Copy link
Contributor

trofi commented Feb 18, 2023

Re-targeted against staging in #216967

@ghost ghost deleted the pr/setup/NIX_OUTPATH_USED_AS_RANDOM_SEED branch February 18, 2023 16:46
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.

2 participants