-
Notifications
You must be signed in to change notification settings - Fork 841
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
stack master fails to build; reporting a shadowed dependency error #4107
Comments
That's not good. I've just closed an issue though (#4084), and that branch of cardano-sl (on ghc-8.2.2, though) does build with dbc74ec, which is 10 commits behind d995208 (including merges). It may be quite straightforward to bisect, especially if that works. Would you be able to test the latter commit? The issue there was due to conflicts in If those suggestions don't help, or if the issue is due to a recent commit, I'll find somebody to take a closer look. |
Ah, I forgot to say: there's a bug in ghc 8.2.1 as described in haskell/cabal#4959 which may be present in 8.0.1 too (I haven't checked whether it was a regression). This might be relevant. |
We're on GHC 8.2.2 for this:
And the exact commit I'm on with the problem:
I'll do a bisect and see what I can find. |
so,
So both the current |
That's frustrating. After some digging, #3554 seems relevant — it and the linked issues refer to GHC ticket 14381 which is a bug in 8.2.2 (fixed in 8.4.4). There should be a workaround in stack, however, and it isn't obvious from that why 1.7.1 works but master does not. One option (which appeared to improve cache behaviour in a similar sounding issue) may be to remove the extra-deps and use a custom resolver — that has been suggested for cardano-sl in the recent past. Meanwhile, @snoyberg and @mgsloan seem to have been the most involved with the shadowed dependency/abi errors, so I'll ask them to take a look. |
I haven't managed to reproduce it yet with clean dirs:
Could you try it with a clean (No need to delete your existing stack root, just passing these two flags with two nonexistent directories; but you do have to |
I did the commands you asked and it built fine. I cleared my Not sure how it got into this state 🤔 |
@parsonsmatt that's good to hear. This should be resolved with documentation of the fix used by @nh2 above; apparently similar issues (shadowed dependencies) have cropped up in IRC. |
I think it would be better if the issue were resolved by identifying how and why the |
I agree on that front. Without a reliable repro, though, it will be all but impossible to confirm that this is fixed. My best bet for improving this situation is #3922, which is a systematic improvement on how a number of the pieces of Stack related to this behave. |
I'm having another instance of this occurring. This time, trying to do:
Doing an Attempting a |
Good to have a reproducible case for this @parsonsmatt. Just to confirm, can you tell me:
|
I did a version bump on LTS for ghc-8.2.2 and saw a similar shadowed dependencies error for the statistics package;
Deleting the following, I'm now able to build statistics on both
|
Thank you, @philderbeast. That clang warning has a fix, by the way (see #4009). |
Yeah, I think we can close until we have another occurrence of this. |
General summary/comments (optional)
stack
master fails to build thecardano-sl
repository on thedevelop
branch right now (commitb7a3334e48f660387fa98202ce413a7cad8c3d57
as of this writing).stack-1.7.1
works fine.Steps to reproduce
For example:
Expected
Actual
gist with verbose output
Stack version
Method of installation
Problem version via
stack upgrade --git
The text was updated successfully, but these errors were encountered: