Skip to content

Commit

Permalink
Work around setup test failure by specifying Cabal >= 1.18
Browse files Browse the repository at this point in the history
For packages with custom setup but without explicit setup deps we
currently specify Cabal >= 1.18 because we have trouble talking to older
Cabal versions, but we don't do anything equivalent for packages with
explicit setup deps.

So in this integration test we had a package specifying simply
  custom-setup
    setup-depends: base, Cabal
and so our solver preference for using installed packages (from the
global db) meant that for older ghc we would pick Cabal-1.16 and then
fail to be able to talk to it.

So we don't solve that problem yet, but work around it in this test by
specifying a constraint. Arguably we should add a constraint or
preference for custom setup script with explicit deps too.
  • Loading branch information
dcoutts committed May 27, 2016
1 parent 17eb3ed commit 38d306f
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ cabal-version: >= 1.10

-- explicit setup deps:
custom-setup
setup-depends: base, Cabal
setup-depends: base, Cabal >= 1.18

library
exposed-modules: A
Expand Down

0 comments on commit 38d306f

Please sign in to comment.