-
Notifications
You must be signed in to change notification settings - Fork 175
Conversation
Great! Could you give or link to some instructions how to compile this? Do we need to build ghc-mod itself with ghc 8.2.2 to be usable with ghc 8.2.2 / stackage lts-10 projects? |
I'm not sure about needing to compile ghc-mod with GHC 8.2.2 / stackage lts-10. But the way I tested it is with GHC 8.2.2 / stackage lts-10 and that's also how I work on my projects.
|
Thank you! Such a stack.yaml file was exactly what I was looking for and the result works perfectly with atom 😃 |
Great, happy to oblige! |
Works for me. |
I can confirm that this compiles and works on Windows 10 x64 Version 1709 Versions:
Tested with Visual Studio Code
|
I can confirm that this worked on Ubuntu 17.04 with no modifications. Versions (with
And now
Thank you @ariskou! I needed |
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.
Cool, we got so tied up making cabal-helper (sorta) work with backpack stuff we forgot to push these changes back to ghc-mod. I could have sworn @alanz had a PR for pretty much this lying around but it seems not.
Anyways great to get some early testing regardless :)
PS: Just FYI you forgot to update the dependency bounds in the cabal file. Never mind, I was looking at the wrong branch.
return ([setupConfigPath distdir], cs) | ||
cs <- runCHQuery $ components $ | ||
GmComponent mempty | ||
CH.<$> ghcOptions |
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.
The CH.
qualifier shouldn't be required here, any particular reason you put those there?
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.
Probably a leftover from trying to get it to compile :) Sorry
I did have this change, and still do, as used in hie. But @DanielG asked me to split the PR and only bring it in once cabal-helper is done. Which is now in limbo. |
As of today, |
What are you talking about? Builds just fine in CI: https://gitlab.com/dxld/cabal-helper/pipelines/16348275 |
Sorry, I mean the |
Try this one https://github.com/alanz/ghc-mod/tree/ghc802-ch8.0 in the meantime. There should be a fuller update soon |
@megafinz Change line 370 in resolveChEntrypoints _ (ChLibEntrypoint em om) = to: resolveChEntrypoints _ (ChLibEntrypoint em om _) = |
Unfortunately, this still doesn't work on Mac OS;
|
@asheshambasta: Have you tried DanielG's suggestion? Works for me. |
Where do I put the compiled ghc-mod then? |
@peterstorm: into any directory that is in your |
@m4lvin Ah yes, figured it out, thank you :) It works now, awesome! |
I have this problem?
Help pls. |
@ayelich See the 2nd post above from @ariskou. You'll also need to apply the change mentioned by @BlackCapCoder to the following file in ghc-mod
|
Hi,
as an extension of the work in PR #911 working on issue #900 for supporting GHC 8.2.x, I have made some small, almost mechanical changes.
These changes in combination with the current
cabal-helper:master
(DanielG/cabal-helper@4bfc6b9) yield a working ghc-mod for GHC 8.2.2, which I have been using for a few weeks now, via the atom plugin.