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
bionic: use fetchzip for everything #123010
Conversation
5ee5f6c
to
ae3baf0
Compare
/rebase master |
The bionic-prebuilt dir thing is new in staging. |
Since the hashes of the remaining two archives that used fetchurl broke, now as good a time as any to switch the remaining fetchurls to fetchzip.
ae3baf0
to
52753af
Compare
I saw this was build by ofborg previously, but for some reason the download doesn't work for me:
|
Interesting, so maybe not only the archive hash is unstable, but its contents as well? |
If you want to look into this:
I think this might be because of the case-insensitive filesystem |
I can reproduce the hashes of your
Not sure what you are getting at with the binary cache, of course I can successfully download already built fixed output derivations… |
Seems like there is indeed a case collision
The |
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.
LGTM
Since the hashes of the remaining two archives that used fetchurl broke,
now as good a time as any to switch the remaining fetchurls to fetchzip.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)