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

rtags: 2.16 -> 2.33 #68234

Closed
wants to merge 2 commits into from
Closed

rtags: 2.16 -> 2.33 #68234

wants to merge 2 commits into from

Conversation

jonringer
Copy link
Contributor

Motivation for this change

fixing broken @ryantm builds

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 nix-review --run "nix-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)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @

$ nix path-info -Sh ./result
/nix/store/g00c4zqpwnbqq9i9jrixmq4b0mq6bwyj-rtags-2.16   955.7M
$ nix path-info -Sh ./result
/nix/store/v3rnjr3ir1x5pdbld7d3hx65ms75y3qx-rtags-2.33   955.7M

@worldofpeace
Copy link
Contributor

@GrahamcOfBorg eval

@mmahut
Copy link
Member

mmahut commented Sep 8, 2019

@GrahamcOfBorg build rtags

1 similar comment
@mmahut
Copy link
Member

mmahut commented Sep 9, 2019

@GrahamcOfBorg build rtags

owner = "andersbakken";
repo = pname;
rev = "v${version}";
sha256 = "10x3zad1sr1ggg66x0rxbn0bb6dvqgj4wc31qgqghwc0ny4by1j2";
Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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.

Copy link
Contributor Author

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 🙃

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