[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] Improved ^c support for gdb/guile
From: |
Eli Zaretskii |
Subject: |
Re: [PATCH v2] Improved ^c support for gdb/guile |
Date: |
Mon, 17 Feb 2014 23:13:57 +0200 |
> Date: Mon, 17 Feb 2014 12:59:22 -0800
> From: Doug Evans <address@hidden>
> Cc: "address@hidden" <address@hidden>, address@hidden
>
> >> +void
> >> +gdbscm_initialize_sigint (void)
> >> +{
> >> + siscm_sigint_pipe[0] = siscm_sigint_pipe[1] = -1;
> >> +
> >> + if (!SCM_USE_PTHREAD_THREADS)
> >> + {
> >> + warning (_("Guile does not have pthreads support."));
> >> + warning (_("Proper SIGINT handling for Guile will be
> >> unavailable."));
> >> + return;
> >> + }
> >
> > The above is what worries me. Guile currently doesn't work in the
> > native MinGW build if configured with threads (it crashes, hangs,
> > etc.). Can't we have decent SIGINT handling without pthreads?
>
> With 2.0.x, no.
> I'm ok with changing the warning, e.g., not printing it at all on
> systems where it would otherwise always be printed, and instead
> documenting the issue for such systems.
>
> The downside is that while Scheme code is running SIGINT is ignored
> (unless one is in the repl, or sets up a SIGINT handler oneself).
Ignored why? because GDB sets the handler to SIG_IGN? Or for some
other reason?
- Re: [PATCH v2] Improved ^c support for gdb/guile, Doug Evans, 2014/02/17
- Re: [PATCH v2] Improved ^c support for gdb/guile,
Eli Zaretskii <=
- Re: [PATCH v2] Improved ^c support for gdb/guile, Doug Evans, 2014/02/17
- Re: [PATCH v2] Improved ^c support for gdb/guile, Ludovic Courtès, 2014/02/18
- Re: [PATCH v2] Improved ^c support for gdb/guile, Eli Zaretskii, 2014/02/18
- Re: [PATCH v2] Improved ^c support for gdb/guile, Ludovic Courtès, 2014/02/18
- Re: [PATCH v2] Improved ^c support for gdb/guile, Eli Zaretskii, 2014/02/18
- Re: [PATCH v2] Improved ^c support for gdb/guile, Ludovic Courtès, 2014/02/18
- Re: [PATCH v2] Improved ^c support for gdb/guile, Eli Zaretskii, 2014/02/18
- Re: [PATCH v2] Improved ^c support for gdb/guile, Ludovic Courtès, 2014/02/18
- MinGW patches, Ludovic Courtès, 2014/02/18
- Re: MinGW patches, Eli Zaretskii, 2014/02/18