Re: bin/48138 (sh(1) wait(1) builtin fails after bg job was SIG(STOP|TSTP|CONT) controlled)

[Steffen Nurpmeso <steffen%sdaoden.eu@localhost>]

The following reply was made to PR bin/48138; it has been noted by GNATS.

From: Steffen Nurpmeso <steffen%sdaoden.eu@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: bin/48138 (sh(1) wait(1) builtin fails after bg job was
 SIG(STOP|TSTP|CONT) controlled)
Date: Sun, 15 Apr 2018 00:01:36 +0200

 Hello Robert Elz!
 
 kre%NetBSD.org@localhost wrote:
  |Synopsis: sh(1) wait(1) builtin fails after bg job was SIG(STOP|TSTP|CONT) \
  |controlled
 
 Oh-ha, yes, i recall something.  (I had to look -- from 2013!)
 
  |State-Changed-From-To: open->feedback
  |State-Changed-By: kre%NetBSD.org@localhost
  |State-Changed-When: Fri, 13 Apr 2018 07:18:23 +0000
  |State-Changed-Why:
  |I believe this is fixed in NetBSD-current (any version after
  |late last October):
 
 This is good to know.
 
  |[jinx]$ /bin/sleep 30 &
  |[jinx]$ jobs
  |[1] + Running                 /bin/sleep 30 &
  |[jinx]$ kill -STOP %1
  |[1] + Suspended (signal)      /bin/sleep 30 &
  |[jinx]$ wait %1
  |[jinx]$ echo $?
  |127
  |[jinx]$ kill -CONT %1
  |[1] + Running                 /bin/sleep 30 &
  |[jinx]$ jobs
  |[1] + Running                 /bin/sleep 30 &
  |[jinx]$ wait %1
  |^C
  |[jinx]$ jobs
  |[1] + Running                 /bin/sleep 30 &
  |[jinx]$ 
  |[jinx]$ 
  |[1]   Done                    /bin/sleep 30 &
  |[jinx]$ 
  |[jinx]$ echo $NETBSD_SHELL
  |20160401
  |
  |That last echo is really just to show that it is the
  |NetBSD /bin/sh that is being used there, the version
  |number hasn't changed since it was added, not all
  |versions that show that have the fix (but any that
  |do not have NETBSD_SHELL at all certainly do not).
  |
  |The fix for this has also been pulled up to NetBSD-8
  |(in general none of the recent sh changes are being
  |pulled up to -7 or -6 so this is not there.)
  |
  |Sorry that I did not have this PR in mind when this was
  |being worked on, or it would have been mentioned in the
  |commit messages.   The fix was a "side effect" (though
  |deliberate) of the fixes for bin/52640 and bin/52641.
 
 I beg you, please!  I was following all that, but a bit
 unreflected it seems.
 
  |If you are able to test, please advise if it seems correct now.
 
 Well.. unfortunately not; not before the 22nd (at 64kBit/s until
 then).  I do not have a usable NetBSD around at the moment, in
 fact, only NetBSD Mail and NetPGP in CVS repositories.  Yes, the
 server and the client both run the very same configuration of
 a very small Linux (with one file, /etc/apk/world, showing
 differences, and the server having an additional file
 /etc/.server).  Ok, there are VMs, but they are very slow and
 occupy the machine, i use them only when i really have to; e.g.,
 compiling libidn2 took two hours in the OpenBSD VM ...
 
 Other than that, it looks perfect :-)
 
 And thanks for the shell glob thread, i knew i have to rewrite the
 shell expression parser so that it creates a tree of inspectable
 objects in order to support ``, $(), ${aXb}, etc., but that i will
 need it to be able to re-apply quoting to strings that will be
 passed to fnmatch(3) i did not realize until yesterday.  Be warned
 you will be credited for that.  (The actual implementation will
 take some time, however, i am with my future g/s-roff fork for the
 forseeable future.)
 
 Greetings from Germany, which enters the five months of autumn
 that disturb the seven months of winter -- next week!  So i wish
 the weekend of anticipation that we have.
 
 --steffen
 |
 |Der Kragenbaer,                The moon bear,
 |der holt sich munter           he cheerfully and one by one
 |einen nach dem anderen runter  wa.ks himself off
 |(By Robert Gernhardt)