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
Fix ar command path in GHC #110695
Fix ar command path in GHC #110695
Conversation
This update was generated by hackage2nix v2.16.0-9-ga573266 from Hackage revision commercialhaskell/all-cabal-hashes@de484d6.
This update was generated by hackage2nix v2.16.0-9-ga573266 from Hackage revision commercialhaskell/all-cabal-hashes@f9bd351.
This update was generated by hackage2nix v2.16.0-9-ga573266 from Hackage revision commercialhaskell/all-cabal-hashes@c1d88e6.
A couple questions:
(Although I guess are all basically the same underlying question.) Can you rebase this on the |
Nice, this should resolve #55995 if I am not mistaken? |
Previously, the "ar command" in the global config of GHC in nixpkgs is simply "ar" instead of a proper absolute path in the nix store. This will result in an "ar: command not found" error when using GHC and cabal in a pure nix shell. This commit adds the patch and applies to all pre-9.0 versions. See output of ghc --info for "ar command" value.
See #55995. tl;dr:
Done. Should I change the PR to target
Yes. |
I was able to reproduce this using the steps explained by @TerrorJack in #110695 (comment). One thing that I had to be careful with is that I have On #85136 (comment), @peti specifically talks about the case where you can get |
@TerrorJack Can you edit this PR to add a comment to link to the upstream PR, and add an explanation of why we can't just pull this from upstream? I'd like to make sure we can track when this patch is no longer needed (so we know when it can be removed from nixpkgs). |
@GrahamcOfBorg build haskell.compiler.ghc865 |
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.
This looks good to me, especially since it seems like they are accepting it as a bug upstream.
I'll wait a week for anyone (like @peti) to comment, but if no one has said anything by then, please feel free to ping me again @TerrorJack and I'll merge this in.
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 think this looks pretty good. Thanks a lot to both of you for your efforts! 👍
Let's wait for the build bot to confirm that everything compiles, and then we should merge so that these patches make into this weeks merge of haskell-updates
into master
.
Well, the test builds don't look so good. That patch is probably not going to make it into |
I've pushed 8fae904 to |
36d8f4b
to
0b62665
Compare
This was merged into Thanks @TerrorJack! |
Previously, the "ar command" in the global config of GHC in nixpkgs is
simply "ar" instead of a proper absolute path in the nix store. This
will result in an "ar: command not found" error when using GHC and cabal
in a pure nix shell. This commit adds the patch and applies to all
pre-9.0 versions.
See output of ghc --info for "ar command" value.
The original patch comes from haskell.nix, see here. Hopefully this will be fixed in upstream 9.x releases.
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)