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
[ghc] fix dylib load command limits #40877
Conversation
See https://phabricator.haskell.org/D4714 for the full details. This will be part of ghc 8.6.
We can probably just drop the linker hack with this, and reenable #37750 (e.g. revert 970f2b5) as well. I'm positive the patch is good for 8.0, and I think for 8.2 as well, for 8.4 we might need to drop the OSiOS. Also 8.4 has logic for In general there is an issue though with using |
Alright, after reconsidering what I just wrote, I did drop Also |
Merged to |
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.
Just waiting for test builds ...
Good stuff! Can we fetchpatch this? |
@Ericson2314 I guess we could try to fetchpatch it, not sure if necessary though? |
@angerman, if we can |
See https://phabricator.haskell.org/D4714 for the full details. This will be part of ghc 8.6. Closes #40877. (cherry picked from commit 4b2ecea) Signed-off-by: Domen Kožar <domen@dev.si>
Motivation for this change
See https://phabricator.haskell.org/D4714 for the full details.
This will be part of ghc 8.6.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)