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

Cabal-2.2.0.0 #3340

Closed
2 of 22 tasks
alexeyzab opened this issue Mar 8, 2018 · 6 comments
Closed
2 of 22 tasks

Cabal-2.2.0.0 #3340

alexeyzab opened this issue Mar 8, 2018 · 6 comments

Comments

@alexeyzab
Copy link
Contributor

alexeyzab commented Mar 8, 2018

Cabal-2.2.0.0 (Mikhail Glushenkov mikhail.glushenkov@gmail.com @23Skidoo) is out of bounds for:

@RyanGlScott
Copy link
Contributor

lens-4.16 revised to allow Cabal-2.2.

(BTW, this might be a bug in stackage-curator: it claims that Cabal is used in the library, executable, test-suite, and benchmark stanzas of lens, but in reality, it's only used in the custom-setup stanza.)

alexeyzab added a commit that referenced this issue Mar 8, 2018
@vshabanov
Copy link

Just uploaded HsOpenSSL-0.11.4.13 that has no upper bound on Cabal.

@DanBurton
Copy link
Contributor

@RyanGlScott I believe this is "working as intended", since the Setup file (custom or not) is indeed what is used to build each of those targets. Therefore any dependencies pulled in by the custom-setup are "used by" all build targets, even though it is not mentioned directly in their corresponding stanzas.

Whether it is worth clarifying the message for this particular corner case is debatable.

@nboldi
Copy link
Contributor

nboldi commented Mar 9, 2018

Fixed in haskell-tools-*-1.0.1.1

tfausak added a commit to tfausak/autoexporter that referenced this issue Mar 11, 2018
DanBurton added a commit that referenced this issue Mar 13, 2018
DanBurton added a commit that referenced this issue Mar 13, 2018
@juhp
Copy link
Contributor

juhp commented Mar 30, 2018

Do we need to keep this open?

@DanBurton
Copy link
Contributor

We do not, since the constraint was removed. Closing.

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

No branches or pull requests

6 participants