Skip to content

Fix test failure for script-extra-dep#4610

Merged
snoyberg merged 1 commit intocommercialhaskell:masterfrom
psibi:integration-failure-extra-deps
Mar 7, 2019
Merged

Fix test failure for script-extra-dep#4610
snoyberg merged 1 commit intocommercialhaskell:masterfrom
psibi:integration-failure-extra-deps

Conversation

@psibi
Copy link
Copy Markdown
Member

@psibi psibi commented Mar 7, 2019

This PR fixes this failure:

Running test script-extra-dep
Running: /home/sibi/.local/bin/stack script.hs

Error: While constructing the build plan, the following exceptions were encountered:

In the dependencies for acme-missiles-0.3:
    stm needed, but the stack configuration has no specified version  (latest matching version is 2.5.0.0)
needed since acme-missiles is a build target.

Some different approaches to resolving this:

  * Consider trying 'stack solver', which uses the cabal-install solver to attempt to find some working build configuration. This can be convenient when dealing with many
    complicated constraint errors, but results may be unpredictable.

  * Recommended action: try adding the following to your extra-deps in /home/sibi/.stack/script/048d6ed9f71bbafeb2cf702cda487d65dbd1010070852364425533166dcd99bc/config.yaml:

stm-2.5.0.0@sha256:1fb8bd117550d560d1b33a6404e27fdb090e70921e0f6434722cdbbce20d8256,2086

Plan construction failed.
Main.hs: Exited with exit code: ExitFailure 1
CallStack (from HasCallStack):
  error, called at ../../../lib/StackTest.hs:52:34 in main:StackTest
  stack, called at ../Main.hs:4:8 in main:Main

stm package is one of the dependency of acme-missiles, so that also
needs to be added as part of extra-dep for the test to run properly

Note: Documentation fixes for https://docs.haskellstack.org/en/stable/ should target the "stable" branch, not master.

Please include the following checklist in your PR:

  • Any changes that could be relevant to users have been recorded in the ChangeLog.md
  • The documentation has been updated, if necessary.

Please also shortly describe how you tested your change. Bonus points for added tests!

This PR fixes this failure:

``` shellsession
Running test script-extra-dep
Running: /home/sibi/.local/bin/stack script.hs

Error: While constructing the build plan, the following exceptions were encountered:

In the dependencies for acme-missiles-0.3:
    stm needed, but the stack configuration has no specified version  (latest matching version is 2.5.0.0)
needed since acme-missiles is a build target.

Some different approaches to resolving this:

  * Consider trying 'stack solver', which uses the cabal-install solver to attempt to find some working build configuration. This can be convenient when dealing with many
    complicated constraint errors, but results may be unpredictable.

  * Recommended action: try adding the following to your extra-deps in /home/sibi/.stack/script/048d6ed9f71bbafeb2cf702cda487d65dbd1010070852364425533166dcd99bc/config.yaml:

stm-2.5.0.0@sha256:1fb8bd117550d560d1b33a6404e27fdb090e70921e0f6434722cdbbce20d8256,2086

Plan construction failed.
Main.hs: Exited with exit code: ExitFailure 1
CallStack (from HasCallStack):
  error, called at ../../../lib/StackTest.hs:52:34 in main:StackTest
  stack, called at ../Main.hs:4:8 in main:Main
```

`stm` package is one of the dependency of acme-missiles, so that also
needs to be added as part of `extra-dep` for the test to run properly
@psibi psibi requested a review from snoyberg March 7, 2019 10:45
@qrilka
Copy link
Copy Markdown
Contributor

qrilka commented Mar 7, 2019

I guess the initial idea was not for GHC 8.2.2 as starting with version 8.4.1 stm is shipped with GHC

@qrilka
Copy link
Copy Markdown
Contributor

qrilka commented Mar 7, 2019

Alternative solution would be to use something depending only on base

@snoyberg snoyberg merged commit 0b701de into commercialhaskell:master Mar 7, 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.

3 participants