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

rambox: fix incorrect sha256 #48606

Closed
wants to merge 4 commits into from
Closed

Conversation

anpryl
Copy link
Contributor

@anpryl anpryl commented Oct 17, 2018

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

* 'master' of https://github.com/NixOS/nixpkgs: (554 commits)
  generators: make toPretty handle floats correctly
  haskell-yesod-core: disable broken test suite
  git-annex: update sha256 hash for the new 6.20181011 version
  cabal2nix: update overrides for latest yaml version
  hackage-packages.nix: automatic Haskell package set update
  hackage2nix: disable failing Hydra builds
  LTS Haskell 12.13
  saga: add mpickering as a co-maintainer after NixOS#39125
  python3Packages.cmd2: 0.9.4 -> 0.9.6
  ghc: add new 8.4.4 version
  apvlv: add .desktop file
  staruml: fix download URL
  spectmorph: init at 0.4.1
  termdown: 1.11.0 -> 1.14.1 (NixOS#48443)
  gotop: 1.2.9 -> 1.5.0 (NixOS#48407)
  emacs-mac: use generic builder
  dolphin-emu: fix metadata (GPLv2 -> GPLv2+, http -> https)
  tests/prometheus-exporters: add new Tor exporter
  dbeaver: 5.2.1 -> 5.2.2
  nixos/hardened: add myself to maintainers
  ...
* 'master' of https://github.com/NixOS/nixpkgs: (104 commits)
  python3.pkgs.pyls: 0.2.1 -> 0.3.0
  chromium: fix aarch64 build (NixOS#48586)
  pythonPackages.cligj: 0.4.0 -> 0.5.0 (NixOS#48584)
  Flexget: 2.15.1 -> 2.16.2 (NixOS#48583)
  gitolite 3.6.7 -> 3.6.10 (NixOS#48562)
  oh-my-zsh: 2018-09-03 -> 2018-09-14
  julia_10: update to 1.0.1
  signal-desktop: 1.16.2 -> 1.16.3
  Revert "Merge pull request NixOS#48122 from zimbatm/pkg-nixos-rebuild"
  terraform-providers.libvirt: fix plugin output (NixOS#48577)
  libssh: 0.7.5 -> 0.7.6
  googler: 3.7 -> 3.7.1 (NixOS#48508)
  chromium plugins: add meta.platforms (NixOS#48495)
  varnish6: 6.0.1 -> 6.1.0 (NixOS#48487)
  erlangR21: 21.0 -> 21.1.1 (NixOS#47371)
  varnish4: 4.1.9 -> 4.1.10 (NixOS#48488)
  tmux: 2.7 -> 2.8
  nixpart0.blivet: fix reference to selinux python modules
  blivet: fix reference to selinux python modules
  magit-filenotify: Fix missing dependency on git
  ...
@anpryl anpryl changed the title Rambox 0.6.1 build fix rambox: fix incorrect sha256 Oct 17, 2018
@lheckemann
Copy link
Member

lheckemann commented Oct 17, 2018

It seems some other commits sneaked their way in here. Also, do you know why the sha256 changed? Perhaps we also need to fix the download URL or something to avoid this happening again. Edit: just saw that it's fetchNodeModules. Not sure how to deal with this, but just updating the sha256 whenever it breaks almost certainly isn't a good approach…

@Mic92
Copy link
Member

Mic92 commented Oct 17, 2018

This is bad it could mean that the npm fixed input derivation is not reproducible, cc @yegortimoshenko

@Mic92
Copy link
Member

Mic92 commented Oct 17, 2018

Cherry-picked here: 4051fc9

@Mic92 Mic92 closed this Oct 17, 2018
@Mic92
Copy link
Member

Mic92 commented Oct 17, 2018

backport: ff7413b

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