darcs

Issue 1091 failure in ctrl-C handling in darcs-unstable

Title failure in ctrl-C handling in darcs-unstable
Priority bug Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, kowey, simonmar, thorkilnaur
Assigned To
Topics

Created on 2008-09-23.01:37:36 by droundy, last changed 2009-10-23.23:36:10 by admin.

Messages
msg6087 (view) Author: droundy Date: 2008-09-23.01:37:34
Hi all,

I've found that if I run

darcs record

and then hit ctrl-C when prompted for the patch name I get the
following error:

Shall I record this change? (1/8)  [ynWsfvpxdaqjk], or ? for help: a
What is the patch name? foodarcs: main thread exited (uncaught exception)

and the lock file is left in _darcs/lock.  This is a pretty serious
regression, and I suspect it relates to the threading code that was
added to deal with I don't recall what silliness we ran into when we
started using ghc with -threaded.  Anyhow, we need to look closely at
that code.

David
msg6088 (view) Author: dagit Date: 2008-09-23.02:00:42
On Mon, Sep 22, 2008 at 6:37 PM, David Roundy <bugs@darcs.net> wrote:
>
> New submission from David Roundy <droundy@darcs.net>:
>
> Hi all,
>
> I've found that if I run
>
> darcs record
>
> and then hit ctrl-C when prompted for the patch name I get the
> following error:
>
> Shall I record this change? (1/8)  [ynWsfvpxdaqjk], or ? for help: a
> What is the patch name? foodarcs: main thread exited (uncaught exception)
>
> and the lock file is left in _darcs/lock.  This is a pretty serious
> regression, and I suspect it relates to the threading code that was
> added to deal with I don't recall what silliness we ran into when we
> started using ghc with -threaded.  Anyhow, we need to look closely at
> that code.

To help narrow the search space, I just tried your example with:
$ darcs --version
2.0.2 (+ 144 patches)

and I get the usual:
Shall I record this change? (2/6)  [ynWsfvpxdaqjk], or ? for help: d
What is the patch name? blah^CwithSignalsHandled: Interrupted!

This is on OSX 10.5.

Jason
msg6092 (view) Author: kowey Date: 2008-09-23.08:20:51
On Mon, Sep 22, 2008 at 19:00:33 -0700, Jason Dagit wrote:
> To help narrow the search space, I just tried your example with:
> $ darcs --version
> 2.0.2 (+ 144 patches)
> 
> and I get the usual:
> Shall I record this change? (2/6)  [ynWsfvpxdaqjk], or ? for help: d
> What is the patch name? blah^CwithSignalsHandled: Interrupted!
> 
> This is on OSX 10.5.

Ah, but this works for me too even with unstable
  darcs 2.0.3pre1 (+ 41 patches)

Maybe it's specific to Linux
msg6096 (view) Author: droundy Date: 2008-09-23.14:56:05
On Tue, Sep 23, 2008 at 09:20:38AM +0100, Eric Kow wrote:
> On Mon, Sep 22, 2008 at 19:00:33 -0700, Jason Dagit wrote:
> > To help narrow the search space, I just tried your example with:
> > $ darcs --version
> > 2.0.2 (+ 144 patches)
> > 
> > and I get the usual:
> > Shall I record this change? (2/6)  [ynWsfvpxdaqjk], or ? for help: d
> > What is the patch name? blah^CwithSignalsHandled: Interrupted!
> > 
> > This is on OSX 10.5.
> 
> Ah, but this works for me too even with unstable
>   darcs 2.0.3pre1 (+ 41 patches)
> 
> Maybe it's specific to Linux

It turns out to triggered by my Workaround refactors that I haven't
yet pushed.  I'm still tracking it down...

David
msg6097 (view) Author: droundy Date: 2008-09-23.15:26:43
Okay, it turns out that in removing the Workaround, I'd accidentally removed the
check to see if we can use System.Posix.Signals... I just checked if
HAVE_SIGNALS was defined without ever defining it.  So this is now fixed.  Silly me.

David
History
Date User Action Args
2008-09-23 01:37:36droundycreate
2008-09-23 02:00:44dagitsetstatus: unread -> unknown
nosy: droundy, kowey, dagit, simonmar, simon
messages: + msg6088
2008-09-23 08:20:54koweysetnosy: droundy, kowey, dagit, simonmar, simon
messages: + msg6092
2008-09-23 14:56:07droundysetnosy: droundy, kowey, dagit, simonmar, simon
messages: + msg6096
2008-09-23 15:26:45droundysetpriority: bug
status: unknown -> resolved
messages: + msg6097
nosy: droundy, kowey, dagit, simonmar, simon
2009-08-06 18:00:28adminsetnosy: + markstos, jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, tommy, beschmi, thorkilnaur, - droundy, simonmar
2009-08-06 21:12:46adminsetnosy: - beschmi
2009-08-10 21:48:53adminsetnosy: + simonmar, - tommy, markstos, darcs-devel, zooko, jast, Serware, mornfall
2009-08-10 23:46:08adminsetnosy: - dagit
2009-08-25 17:25:14adminsetnosy: + darcs-devel, - simon
2009-08-27 14:12:12adminsetnosy: kowey, darcs-devel, simonmar, thorkilnaur, dmitry.kurochkin
2009-10-23 22:37:42adminsetnosy: + marlowsd, - simonmar
2009-10-23 23:36:10adminsetnosy: + simonmar, - marlowsd