patch to call release_pgrp() on exit

patch to call release_pgrp() on exit

[Danek Duvall]

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

This patch was posted in 19140, but seems not to have been applied.
It's needed to get exiting from su working properly (without it, it
looks like the su shell is suspended, but foregrounding it makes it go
away).

I know there was some discussion at the time of how necessary this patch
was, but unless someone can point me to the specific misbehavior of
Solaris su, I'll need to include this patch for the Solaris build of
zsh (as I did for 4.1.1).

Can it make it into 4.2?

Thanks,
Danek

[-- Attachment #2: pgrp.patch --]
[-- Type: text/plain, Size: 327 bytes --]

--- Src/builtin.c.orig	Fri Sep 26 11:38:52 2003
+++ Src/builtin.c	Fri Sep 26 11:39:19 2003
@@ -3977,6 +3977,9 @@
     if (sigtrapped[SIGEXIT])
 	dotrap(SIGEXIT);
     runhookdef(EXITHOOK, NULL);
+    if (opts[MONITOR] && interact && (SHTTY != -1)) {
+	release_pgrp();
+    }
     if (mypid != getpid())
 	_exit(val);
     else

Re: patch to call release_pgrp() on exit

[Peter Stephenson]

Danek Duvall wrote:
> This patch was posted in 19140, but seems not to have been applied.
> It's needed to get exiting from su working properly (without it, it
> looks like the su shell is suspended, but foregrounding it makes it go
> away).

I've applied it.

-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR Ltd., Science Park, Milton Road,
Cambridge, CB4 0WH, UK                          Tel: +44 (0)1223 692070


**********************************************************************
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they
are addressed. If you have received this email in error please notify
the system manager.

This footnote also confirms that this email message has been swept by
MIMEsweeper for the presence of computer viruses.

www.mimesweeper.com
**********************************************************************

Re: patch to call release_pgrp() on exit

[Danek Duvall]

On Tue, Mar 16, 2004 at 11:08:47AM +0000, Peter Stephenson wrote:

> I've applied it.

Thanks!