-
-
Notifications
You must be signed in to change notification settings - Fork 14.9k
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
idrisPackages: Fix linking to gmp library and cc #58319
Conversation
This has been broken since 5d18129, which updated idris from 1.3.0 to 1.3.1, which included idris-lang/Idris-dev#4472 as the cause of the error. I'm still not entirely sure why this broke it though. This now way should be rather future proof, it uses NIX_CFLAGS to pass gpm link flags to our CC wrapper directly. The `NIX_CC_WRAPPER_${stdenv.cc.infixSalt}_TARGET_HOST` part I'm pretty sure is needed for the CC wrapper to know that those CFLAGS are meant for the cc running on the HOST.
This should make it work on Darwin with clang.
12e7c7a
to
271403c
Compare
@GrahamcOfBorg build idrisPackages.contrib |
Okay well that's weird. I tried building @GrahamcOfBorg build idrisPackages.effects idrisPackages.glfw idrisPackages.html idrisPackages.http idrisPackages.idrisscript idrisPackages.pipes idrisPackages.webgl idrisPackages.yaml And also some of the currently failing ones on Darwin: @GrahamcOfBorg build idrisPackages.bi idrisPackages.config idrisPackages.electron idrisPackages.eternal idrisPackages.ipkgparser idrisPackages.mhd |
State of builds:
|
@GrahamcOfBorg build idrisPackages.contrib |
1 similar comment
@GrahamcOfBorg build idrisPackages.contrib |
0c4e596
to
4dac0da
Compare
This closes #54579 |
@GrahamcOfBorg build idrisPackages.contrib |
4dac0da
to
271403c
Compare
Doesn't solve the darwin issue.. I have no idea what's up with that tbh. Maybe something during the compilation of Prelude failed, making DecEq unavailable, dunno This is the error for future reference:
I think I'll just merge this for now, idris currently is in an almost unusable state, so anything is an improvement. |
Not sure what to do about this right now. There's clearly a regression here, on my mac idrisPackages.contrib doesn't build anymore, but no idea where it comes from. |
you may get some insight from #idris or perhaps create an issue |
I'll just merge this now, idris is almost unusable without this |
Seems to build just fine on macOS Mojave, but not on High Sierra. So this problem should automatically disappear as hydra and people update their version |
Been playing around with Idris and notice that this issue seem to affect 19.03, would any mind if I backport this? Seems to work well for me. |
@rycee Sounds good |
Cool, done 🙂 |
Fixes #54579 and probably #52796
Ping @ar1a @marsam @shmish111 @mpickering
This has been broken since 5d18129,
which updated idris from 1.3.0 to 1.3.1, which included
idris-lang/Idris-dev#4472 as the cause of the
error. I'm still not entirely sure why this broke it though.
This way should be rather future proof, it uses NIX_CFLAGS to pass
gpm link flags to our CC wrapper directly. The
NIX_CC_WRAPPER_${stdenv.cc.infixSalt}_TARGET_HOST
part I'm pretty sureis needed for the CC wrapper to know that those CFLAGS are meant for the
cc running on the HOST.
The second commit just switches to stdenv's cc.
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)