Skip to content
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

Initialize GHC plugins #224

Merged
merged 1 commit into from
Apr 26, 2019
Merged

Conversation

leonschoorl
Copy link
Contributor

Since GHC-8.6 plugins need to be initialized on a per module basis.
Without this any modules using plugins fail to compile inside doctest.

This code is adapted from how this was fixed in haddock:
https://github.com/haskell/haddock/pull/983/files#diff-a6119dd2da927bbff2c39eda86cdc733

I have a testcase which I used to verify this is working at: https://github.com/leonschoorl/doctest/blob/load-plugins-test/TestPlugins.hs
But it requires the ghc-typelits-knownnat package to be installed, and I don't know how to integrate this into the testsuite.

Since GHC-8.6 plugins need to be initialized on a per module basis.
Without this any modules using plugins fail to compile inside doctest.

This code is adapted from how this was fixed in haddock:
https://github.com/haskell/haddock/pull/983/files#diff-a6119dd2da927bbff2c39eda86cdc733
@quasicomputational
Copy link
Collaborator

Seems sensible enough and Travis likes it. I assume that if this breaks anyone, they'll come here and complain, so let's go for it.

@quasicomputational quasicomputational merged commit bcc7492 into sol:master Apr 26, 2019
quasicomputational added a commit to quasicomputational/doctest that referenced this pull request Apr 26, 2019
quasicomputational added a commit that referenced this pull request Apr 26, 2019
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.

None yet

2 participants