stty: fix sane setting of susp to ^z on Solaris

* src/stty.c: Disable setting of "swtch" to ^z as that
conflicts with and disables using ^z as "susp".
* NEWS: Mention the bug fix.
Reported and tested by Rich Burridge at:
http://bugs.gnu.org/23866

diff --git a/NEWS b/NEWS
index bfaf2399e0001a4659cc81818150c727efc5e4e8..e03952e85d0167ad9d3c17cc6bf868fa33cd2bab 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -24,6 +24,9 @@ GNU coreutils NEWS                                    -*- outline -*-
   stty --help no longer outputs extraneous gettext header lines
   for translated languages. [bug introduced in coreutils-8.24]
 
+  stty "sane" again sets "susp" to ^z on Solaris, and leaves "swtch" undefined.
+  [This bug previously fixed only on some older Solaris systems]
+
   seq now immediately exits upon write errors.
   [This bug was present in "the beginning".]
 

diff --git a/src/stty.c b/src/stty.c
index 44cdcddc685fcf1e59f5e1469cb93afa73a47826..467f8ae89ca323ae213dc4b32c22ca23bbd22f88 100644 (file)
--- a/src/stty.c
+++ b/src/stty.c
@@ -118,9 +118,9 @@
 # define CSWTCH _POSIX_VDISABLE
 #endif
 
-/* SunOS 5.3 loses (^Z doesn't work) if 'swtch' is the same as 'susp'.
+/* 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