Re: Subshell exiting, suspend problem

[Philippe Troin <phil@fifi.org>]

[-- Attachment #1: Type: text/plain, Size: 1432 bytes --]

Philippe Troin <phil@fifi.org> writes:

> Danek Duvall <duvall@emufarm.org> writes:
> 
> > On Fri, Oct 03, 2003 at 03:24:43PM -0700, Danek Duvall wrote:
> > 
> > > On Fri, Oct 03, 2003 at 03:06:09PM -0700, Philippe Troin wrote:
> > > 
> > > > Danek Duvall <duvall@emufarm.org> writes:
> > > >
> > > > > However, I'm now unable to suspend vi or ncftp, or, I would guess,
> > > > > any curses program.
> > > > 
> > > > That's strange: this patch only changes zsh's exit behavior.
> > > 
> > > Indeed.  I'll recompile both the patched version and the unpatched
> > > version from scratch and see if I can reproduce all this behavior.
> > 
> > Mea culpa.  I'd run configure in the background, and it failed to detect
> > tcsetpgrp properly.  Rebuilding in the foreground solved the issue.
> 
> Glad to hear that :-)
> 
> I'll see if I can fix the configure script for that case. At worst, I
> can make it fail if ran in the background.

Danek,

Can you check that you can reproduce the bit about failing to detect
tcsetpgrp properly?

If you can reproduce it, can you then try this patch and see if it
fixes the configure problem? You will need to regenerate the configure
script by running "autoconf" after patching zshconfig.ac.

BTW, I could not reproduce your problem at all. If I run configure in
the background, it just stops during the "checking if tcsetpgrp()
actually works..." test. This patch fixes this part.

Thanks,
Phil.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: zsh-tcsetgrp-bgconfigure.patch --]
[-- Type: text/x-patch, Size: 700 bytes --]

Index: zshconfig.ac
===================================================================
RCS file: /cvsroot/zsh/zsh/zshconfig.ac,v
retrieving revision 1.41
diff -b -u -r1.41 zshconfig.ac
--- zshconfig.ac	15 Sep 2003 09:20:21 -0000	1.41
+++ zshconfig.ac	17 Oct 2003 16:47:40 -0000
@@ -1614,6 +1614,7 @@
 dnl for instance, BeOS R4.51 does not support it yet
 dnl -----------
 if test -t 0 && test $ac_cv_func_tcsetpgrp = yes; then
+    trap "" SIGTTOU
     AC_CACHE_CHECK(if tcsetpgrp() actually works,
     zsh_cv_sys_tcsetpgrp,
     [AC_TRY_RUN([
@@ -1631,6 +1632,7 @@
     if test $zsh_cv_sys_tcsetpgrp = no; then
       AC_DEFINE(BROKEN_TCSETPGRP)
     fi
+    trap - SIGTTOU
 fi
 
 dnl -----------