bug#23866: stty sane behaviour.

[Pádraig Brady]

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.