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

Assertion failure in D.C.Install #3912

Closed
23Skidoo opened this issue Sep 29, 2016 · 5 comments
Closed

Assertion failure in D.C.Install #3912

23Skidoo opened this issue Sep 29, 2016 · 5 comments
Assignees
Milestone

Comments

@23Skidoo
Copy link
Member

With current master, when I do:

$ cabal get log
$ cd log-0.5.3
$ cabal sandbox init
$ cabal install --only-dep

I get the following assertion failure:

$ cabal install --only-dep
Warning: the --global flag is deprecated -- it is generally considered a bad
idea to install packages into the global store
Resolving dependencies...
Notice: installing into a sandbox located at
/home/refold/code/haskell/log-0.5.3/.cabal-sandbox
Configuring base64-bytestring-1.0.0.1...
Configuring base-orphans-0.5.4...
[...]
Building nats-1.1.1...
Installed nats-1.1.1
Configuring network-2.6.3.1...
Configuring old-locale-1.0.0.7...
Assertion failed
CallStack (from HasCallStack):
assert, called at ./Distribution/Client/Install.hs:1427:34 in
main:Distribution.Client.Install

This does not seem to happen with 1.24 (I checked that I compiled with assertions on).

@23Skidoo 23Skidoo added this to the 2.0 milestone Sep 29, 2016
@23Skidoo
Copy link
Member Author

/cc @ezyang - The assertion was introduced by you in 3601819.

@ezyang
Copy link
Contributor

ezyang commented Sep 29, 2016

Can you post a -v3 log with -j1 (no parallelism).

@ezyang ezyang self-assigned this Sep 29, 2016
@23Skidoo
Copy link
Member Author

Uploaded the -j1 -v3 log here: https://gist.github.com/23Skidoo/70445e3c9d4de15165f2c21a789ecca9

@ezyang
Copy link
Contributor

ezyang commented Sep 29, 2016

Ah, that makes this very clear. hsc2hs indeed, does not have a library. But why are we building it in the legacy codepath? I thought I fixed this in 9e99b3f but if you are on the newest master, clearly not!

@ezyang
Copy link
Contributor

ezyang commented Oct 1, 2016

Confirmed reproduction: when I do your test case with HEAD hsc2hs gets built.

ezyang added a commit to ezyang/cabal that referenced this issue Oct 2, 2016
In 9e99b3f I turned of executable
solving for legacy configure, but I forgot to turn it off for legacy
install too, which lead to assertion failures.

Fixes haskell#3912.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
ezyang added a commit to ezyang/cabal that referenced this issue Oct 2, 2016
In 9e99b3f I turned of executable
solving for legacy configure, but I forgot to turn it off for legacy
install too, which lead to assertion failures.

Fixes haskell#3912.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants