On 29/06/16 02:06, Rich Burridge wrote: > On 06/28/2016 05:05 PM, Pádraig Brady wrote: >> On 29/06/16 00:03, Rich Burridge wrote: >>> ... >> Thanks for the detailed analysis. >> It looks like this was already handled and we need to >> expand the conditions where this is done. The current code is: >> >> /* SunOS 5.3 loses (^Z doesn't work) if 'swtch' is the same as 'susp'. >> So the default is to disable 'swtch.' */ >> #if defined __sparc__ && defined __svr4__ >> # undef CSWTCH >> # define CSWTCH _POSIX_VDISABLE >> #endif >> >> How about we guard this with just: >> >> #ifdef __sun >> #endif > > Assuming you mean: > > --- src/stty.c.orig 2016-06-28 17:48:25.580754994 -0700 > +++ src/stty.c 2016-06-28 17:51:21.725276863 -0700 > @@ -120,7 +120,7 @@ > > /* SunOS 5.3 loses (^Z doesn't work) if 'swtch' is the same as 'susp'. > So the default is to disable 'swtch.' */ > -#if defined __sparc__ && defined __svr4__ > +#if defined __sun > # undef CSWTCH > # define CSWTCH _POSIX_VDISABLE > #endif > > and I'm sure you do, this works great.
Cool, pushed at: http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=v8.25-41-gdfae782 thanks, Pádraig.
