Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

ghc: install static gmp in prefix #39025

Closed
wants to merge 1 commit into from

Conversation

geoff-nixon
Copy link
Contributor

The static GMP built for ghc is presently installed in the buildpath; additionally, the arguments to configure were incorrect. As the library is still needed for static linking Haskell applications which make use of the bignum functionality, install into libexec instead.

See line notes in #38963.

@geoff-nixon
Copy link
Contributor Author

Edit: I've now in addition used the proposed new method in #39026 to re-enable the jobserver. I can revert if there are objections. Changed to a "-j1" per discussion in #39026.

@geoff-nixon geoff-nixon force-pushed the ghc-gmp-workaround branch 2 times, most recently from 66037aa to 670b0ab Compare April 25, 2015 08:46
@MikeMcQuaid
Copy link
Member

Looks good, thanks 👍

@MikeMcQuaid
Copy link
Member

Probably wants a revision bump too here so we can assume everyone using ghc has their gmp sorted.

@MikeMcQuaid
Copy link
Member

@geoff-codes Can you add a revision bump? Thanks!

@geoff-nixon
Copy link
Contributor Author

@MikeMcQuaid I'm on it. It turns out there are a couple more outstanding issues I'm testing now.

end

system "make"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need to make again if we're not building tests.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@geoff-nixon
Copy link
Contributor Author

Ok, so I've pushed a rev 2 (with a few more changes). Hopefully they're not too objectionable.
Note I'll probably have another PR for you in a day for too with more significant changes, but I don't think it will require another rev bump if/when they're merged.

@geoff-nixon
Copy link
Contributor Author

Well this is part of what I was worried about. Hold tight...

@MikeMcQuaid
Copy link
Member

Closing in favour of #39134.

@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants