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-install head ignores sigint (ctr+c) but responds to sigsusp (ctrl+z), sometimes (don't have a reproducable test case yet) #1345

Closed
cartazio opened this issue May 23, 2013 · 7 comments

Comments

@cartazio
Copy link
Contributor

i'm testing out using cabal-install head (head as of now, so hash 2f32275) sandboxing, and i've had an instance of ctrl+c not killing cabal in a sandboxed build, but ctrl+z suspending it correctly

(still trying to figure out how to reproduce this problem though, so this may be a useless ticket if i cant reproduce )

@23Skidoo
Copy link
Member

Probably happens when we're reinstalling dependencies. This is the same behaviour that cabal-install has (#971).

@dcoutts
Copy link
Contributor

dcoutts commented Oct 17, 2013

It may also be due to the fact that we're using System.Process API that does not do let us do ctl-c handling correctly yet. The old System.Cmd.system did this right, but that feature has been hard to expose with the more flexible System.Process API. There's a ghc ticket on this if someone wants to work on it.

@23Skidoo
Copy link
Member

I wonder whether this has been fixed by #1448.

@gracjan
Copy link

gracjan commented Mar 2, 2015

Yes, #1448 strikes again. This is super annoying. How come that it returned?

@jsl
Copy link

jsl commented Mar 3, 2015

I just tested this issue with Cabal 1.22.0. Results:

  • Ctrl-C does kill cabal install --only-dependencies --enable-tests, so that is working correctly
  • Ctrl-C does kill cabal build, so that is working correctly

@gracjan I responded to your comment on #1448. It would help to get some more info on that ticket since I can't reproduce the issue you mentioned with a current cabal.

In the meantime, @cartazio it seems like the issue in this ticket has probably been resolved, since I can kill cabal with Ctrl-C during builds and installs on a recent version. Would you mind closing this ticket unless this is something you're still experiencing?

Thanks a lot!

@23Skidoo
Copy link
Member

23Skidoo commented Mar 3, 2015

It likely depends on the version of process cabal-install was compiled with. Old versions don't have the delegate_ctlc feature, but we still support them.

@23Skidoo
Copy link
Member

23Skidoo commented Mar 3, 2015

Optimistically closing.

@23Skidoo 23Skidoo closed this as completed Mar 3, 2015
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

5 participants