Skip to content

Conversation

abooij
Copy link
Contributor

@abooij abooij commented Oct 30, 2016

This is supposed to test if a foreign library can depend on Haskell code in the same project but in a different binary object.

@mention-bot
Copy link

@abooij, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edsko to be a potential reviewer.

@ezyang
Copy link
Contributor

ezyang commented Oct 31, 2016

Sorry, this PR got swizzled by the package-tests move. Just transplant the changes into the new cabal-testsuite directory, or tell me if you want me to do it.

@ezyang
Copy link
Contributor

ezyang commented Oct 31, 2016

Lovely, nice new test which uncovers some bugs in the foreign library implementation.

  1. Run update-cabal-files.sh, that will fix the first CI failure
  2. For some reason foreign libraries with an external Haskell library dep don't work for GHC 7.6 and earlier. I'm happy to bump the support window of foreign libraries to 7.8+, but then checkForeignLibSupported in Cabal/Distribution/Simple/Configure.hs has to be updated, and the test suites configured to not run in that case.

Also bump version requirements for foreign libs (ghc>=7.8)
@abooij
Copy link
Contributor Author

abooij commented Oct 31, 2016

I made the modifications that you proposed: rebased, ran update-cabal-files.sh, and bumped to 7.8 in Configure.hs and Check.hs.

@ezyang ezyang merged commit cb1364b into haskell:master Oct 31, 2016
@ezyang
Copy link
Contributor

ezyang commented Oct 31, 2016

Pinging @edsko: If you need foreign libraries to work on pre GHC 7.8, you better check this bug out!

@edsko
Copy link
Contributor

edsko commented Nov 1, 2016

No, pre 7.8 support for limited anyway (#2540 (comment)). Fine with me to just regard as unsupported.

abooij added a commit to abooij/cabal that referenced this pull request Nov 1, 2016
erikd pushed a commit to erikd/cabal that referenced this pull request Jan 28, 2017
Also bump version requirements for foreign libs (ghc>=7.8)
erikd pushed a commit to erikd/cabal that referenced this pull request Jan 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants