From: Sven Wischnowsky <wischnow@informatik.hu-berlin.de>
To: zsh-workers@sunsite.dk
Cc: Timothy Miller <tsm@cs.brown.edu>
Subject: Re: fatal flaw zsh 4.0.1 on irix 6.3 & 6.5: suspend "ls -l|less" then resume hangs
Date: Tue, 31 Jul 2001 10:20:57 +0200 (MET DST) [thread overview]
Message-ID: <200107310820.KAA02935@beta.informatik.hu-berlin.de> (raw)
In-Reply-To: <yeh3d7egjdz.fsf@water.cs.brown.edu>
Timothy Miller wrote:
> ...
>
> Ah. He didn't send them to me (just the list, from examining the headers),
> so of course I didn't get or try them.
Sorry.
> exec.c patch:
>
> The line number given in this patch is 9 more than the corresponding line in
> the zsh-4.0.2 source file I have, just as a note.
>
> Incidentally, according to the irix man page for killpg, you're supposed to
> #define _BSD_SIGNALS or _BSD_COMPAT before signal.h to use it; zsh appears
> to use BSD_SIGNALS (not _BSD_SIGNALS) in configure. However, it still uses
> BSD_SIGNALS in -pws-2. Further, if I take the cc command line for exec.c,
> change cc to gcc, -c to -E, remove -o exec.o, and add -dD to get the macro
> definitions in place, then it looks like none of these *BSD* symbols are
> defined anywhere. The man page also says using BSD and SYSV facilities in
> the same program is strongly discouraged and will result in unpredictable
> behavior, in case you weren't aware (I don't know if zsh does that).
Aha. Hm, well, is there any Unix where killpg(p, s) does something
different than kill(-p, s)? If not, we could either change all places
where we use killpg() or just add a killpg macro to zsh.h or signals.h.
> ...
>
> So it looks like this patch fixes the hanging problem, except that cat still
> isn't stopped (it is when I do the same test under solaris with 4.0.1).
Yes, it's a lot better already. You could test if modifying the line I
changed to use kill() instead of killpg() to:
kill(-jobtab[list_pipe_job].gleader, 0) == -1) {
fixes the problem with cat not being stopped. (I've only inserted a `-'
before the `jobtab[...]' here.)
Bye
Sven
--
Sven Wischnowsky wischnow@informatik.hu-berlin.de
next prev parent reply other threads:[~2001-07-31 8:21 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-06-11 15:11 Timothy Miller
2001-06-12 8:15 ` Sven Wischnowsky
2001-06-12 15:00 ` Timothy Miller
[not found] ` <1010612154026.ZM24394@candle.brasslantern.com>
2001-06-12 16:13 ` Timothy Miller
2001-06-13 3:55 ` Bart Schaefer
2001-07-06 22:40 ` Timothy Miller
2001-06-14 7:32 ` Sven Wischnowsky
2001-07-06 22:52 ` Timothy Miller
2001-07-10 15:22 ` Timothy Miller
2001-07-10 16:25 ` Bart Schaefer
2001-07-10 21:26 ` Timothy Miller
2001-07-10 23:27 ` Bart Schaefer
2001-07-11 8:33 ` Sven Wischnowsky
2001-07-11 9:51 ` Sven Wischnowsky
2001-07-11 18:59 ` Timothy Miller
2001-07-12 3:39 ` Bart Schaefer
2001-07-30 20:18 ` Timothy Miller
2001-07-30 21:24 ` Bart Schaefer
2001-07-30 22:08 ` Timothy Miller
2001-07-31 8:20 ` Sven Wischnowsky [this message]
2001-07-31 8:30 ` Borsenkow Andrej
2001-07-31 14:35 ` Timothy Miller
2001-07-31 20:50 ` Timothy Miller
2001-08-01 8:35 ` Sven Wischnowsky
2002-02-25 9:31 ` Sven Wischnowsky
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200107310820.KAA02935@beta.informatik.hu-berlin.de \
--to=wischnow@informatik.hu-berlin.de \
--cc=tsm@cs.brown.edu \
--cc=zsh-workers@sunsite.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).