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
rtags: 2.16 -> 2.33 #68234
rtags: 2.16 -> 2.33 #68234
Conversation
@GrahamcOfBorg eval |
@GrahamcOfBorg build rtags |
1 similar comment
@GrahamcOfBorg build rtags |
owner = "andersbakken"; | ||
repo = pname; | ||
rev = "v${version}"; | ||
sha256 = "10x3zad1sr1ggg66x0rxbn0bb6dvqgj4wc31qgqghwc0ny4by1j2"; |
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.
It appears we are getting a hash mismatch on darwin
wanted: sha256:10x3zad1sr1ggg66x0rxbn0bb6dvqgj4wc31qgqghwc0ny4by1j2
got: sha256:0xsds7xia2dwcf2v9cac7wfbvx6p0h0c0yl7w6ha3pfddwawabsc
The fetcher we're actually using in fetchFromGitHub
is fetchgit
because of the submodules.
I believe this is because normalization the darwin filesystem does to certain files, if we used an archive and got the hash without unpacking this wouldn't be an issue.
Perhaps we can do
if stdenv.isDarwin then "0xsds7xia2dwcf2v9cac7wfbvx6p0h0c0yl7w6ha3pfddwawabsc"
else "10x3zad1sr1ggg66x0rxbn0bb6dvqgj4wc31qgqghwc0ny4by1j2"
and add a comment. Updating this will be slightly more difficult though.
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.
another alternative would be to "fetch" the submodule myself, and just link the src before configuring. I tried this at first and hit a bump, so i just went with the fetchSubmodules option.
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.
Actually, can you revert to how it was fetched before?
below had
# unicode file names lead to different checksums on HFS+ vs. other
# filesystems because of unicode normalisation
postFetch = ''
rm $out/src/rct/tests/testfile_*.txt
'';
which fixed the exact issue I mentioned.
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.
i kicked the can and added rct as a proper package, however, rct now might fail to eval 🙃
82299dd
to
4620c94
Compare
Motivation for this change
fixing broken @ryantm builds
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @