-
Notifications
You must be signed in to change notification settings - Fork 204
Use new cabal-helper #1245
Use new cabal-helper #1245
Conversation
Relied on old verison of cabal-helper
|
@alanz this is all using your work on new-cabal-helper with ghc-mod, should I create a PR to merge this into your fork? |
| d ^. source `shouldBe` Just "ghcmod" | ||
| d ^. message `shouldBe` | ||
| (T.pack "readCreateProcess: stack \"build\" \"--only-configure\" \".\" (exit 1): failed\n") | ||
| (T.pack "callProcessStderr: stack --stack-yaml=/root/build/test/testdata/badProjects/cabal/stack.yaml build --only-configure . (exit 1): failed\n") |
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.
For the HaRe tests I copy certain projects to $TEMPDIR/testproj, and then work with that. This looks like a very brittle hard coded path.
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.
I never even noticed the path, I just copied and pasted it verbatim. Will do a fuzzier expectation
|
While the stability seems to be improved in general, it reopens #1264, probably since the patch for it was submitted after this pr had been opened. |
|
When you open |
|
What's missing for this to be merged? I see that the cabal job has passed, but the others are failing with some I'd love to help in order to get this merged |
|
The cabal job is passing just because the cabal job doesn’t run any tests 😬
The first thing would be to probably get the remaining tests passing locally, and from what I can remember there were some new problems when working with cabal projects nested within each other, like @fendor pointed out above with the hie-plugin-api sub-project in haskell-ide-engine itself.
PRs to this branch are more than welcome! :)
… On 10 Sep 2019, at 16:53, Nick Tchayka ***@***.***> wrote:
What's missing for this to be merged? I see that the cabal job has passed, but the others are failing with some allow-newer errors. I guess this is due to the version bump of cabal-helper?
I'd love to help in order to get this merged
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
|
This is done in hie-bios |
This removes cabal-helper-wrapper, and so will open up the way for static binaries until hie-bios lands.
Also noticeably, it removes the build plugin since it relied on the old cabal-helper API: I plan on removing this in #1105 anyway as it is only used by the old JSON transport.
Please dogfood this a bit!