Skip to content

Conversation

@hvr
Copy link
Member

@hvr hvr commented Oct 18, 2016

Unfortunately, "native" GHC advertises support for JavaScriptFFI even though
it doesn't support it. See also https://ghc.haskell.org/ticket/11214 for respective
bug.

However, in order to properly declare that packages require JavaScriptFFI support
via other-extensions we need to fixup the list of extensions fed to the cabal solver.

This patch does something similiar to the workaround we added some time ago to
filter out TemplateHaskell for older GHCs which didn't properly advertise
TemplateHaskell availability (c.f. 9f68eb4)

Unfortunately, "native" GHC advertises support for `JavaScriptFFI` even though
it doesn't support it. See also https://ghc.haskell.org/ticket/11214 for respective
bug.

However, in order to properly declare that packages require `JavaScriptFFI` support
via `other-extensions` we need to fixup the list of extensions fed to the cabal solver.

This patch does something similiar to the workaround we added some time ago to
filter out TemplateHaskell for older GHCs which didn't properly advertise
`TemplateHaskell` availability (c.f. 9f68eb4)
@mention-bot
Copy link

@hvr, thanks for your PR! By analyzing the history of the files in this pull request, we identified @23Skidoo, @phadej and @dcoutts to be potential reviewers.

@hvr
Copy link
Member Author

hvr commented Oct 18, 2016

@23Skidoo this patch should be cherry-picked into the upcoming 1.24 point release as well

I've tested the patch locally with ghcjs to make sure it does the right thing in the presence of other-extensions: JavaScriptFFI

/cc @luite

@ezyang
Copy link
Contributor

ezyang commented Oct 18, 2016

OK looks fine. A bit annoying that we can't test.

@ezyang ezyang added this to the 1.24.0.1 milestone Oct 18, 2016
Copy link
Member

@23Skidoo 23Skidoo left a comment

Choose a reason for hiding this comment

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

LGTM.

@23Skidoo 23Skidoo merged commit 458e2d7 into haskell:master Oct 18, 2016
@23Skidoo
Copy link
Member

Merged, thanks! Also cherry-picked into 1.24 (0bae317).

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