[8057] do not check for stty

[gavinsmith0123]

Revision: 8057
          http://svn.sv.gnu.org/viewvc/?view=rev&root=texinfo&revision=8057
Author:   gavin
Date:     2018-08-14 05:17:22 -0400 (Tue, 14 Aug 2018)
Log Message:
-----------
do not check for stty

Modified Paths:
--------------
    trunk/ChangeLog
    trunk/info/t/Init-test.inc

Modified: trunk/ChangeLog
===================================================================
--- trunk/ChangeLog     2018-08-14 08:03:17 UTC (rev 8056)
+++ trunk/ChangeLog     2018-08-14 09:17:22 UTC (rev 8057)
@@ -1,5 +1,11 @@
 2018-08-14  Gavin Smith  <address@hidden>
 
+       * info/t/Init-test.inc: Do not check whether the terminal needs 
+       to be reset with stty, because the command that caused this 
+       problem ("read -t") is not being used any more.
+
+2018-08-14  Gavin Smith  <address@hidden>
+
        * tp/t/01use.t: Inline file tp/t/manual_tree.pl.
 
 2018-08-13  Gavin Smith  <address@hidden>

Modified: trunk/info/t/Init-test.inc
===================================================================
--- trunk/info/t/Init-test.inc  2018-08-14 08:03:17 UTC (rev 8056)
+++ trunk/info/t/Init-test.inc  2018-08-14 09:17:22 UTC (rev 8057)
@@ -57,10 +57,6 @@
 # Make sure that non-interactive operation works even if terminal is dumb
 TERM=dumb; export TERM
 
-# For interactive tests, we try to set this in Init-inter.inc
-STTY=true
-reset_required=no
-
 # Clean up if the test is interrupted, for example if the user types
 # C-c, to avoid lingering child processes.  Signal 2 is SIGINT.
 trap cleanup 2
@@ -74,12 +70,6 @@
   rm -f $GINFO_OUTPUT
   rm -f $PIPEIN $PTY_TYPE
   
-  # We do this because some versions of ksh93 (including that in
-  # Solaris 11) have a bug where a timed-out "read -t" can mess up
-  # the terminal settings, leading to characters not being echoed.
-  # This is only done if we don't start with "-echo" at the beginning.
-  test $reset_required = yes && { $STTY | grep '[-]echo' ; } && $STTY sane
-
   #killall `basename $0`  # see below
   exit $RETVAL
 }
@@ -151,10 +141,6 @@
   # Avoid ginfo complaining that terminal is too dumb
   TERM=vt100; export TERM
   
-  # See 'cleanup' function
-  findprog stty && { STTY=stty ; reset_required=yes ; }
-  test $reset_required=yes && { $STTY | grep [-]echo ; } && reset_required=no
-  
   # Create named pipes to communicate with pseudotty program, or quit.
   rm -f $PIPEIN $PTY_TYPE # must already be defined
   if findprog mkfifo; then
@@ -194,11 +180,11 @@
 run_ginfo ()
 {
   init_interactive_test
+
   (
     exec 7>&- ; # Close fd from parent shell
-
-    exec $GINFO $GINFO_OPTIONS "$@" 0<>$PTS_DEVICE 1<&0 ; ) &
-
+    exec $GINFO $GINFO_OPTIONS "$@" 0<>$PTS_DEVICE 1<&0 ;
+  ) &
   GINFO_PID=$!
   (sleep 3 ; kill $GINFO_PID) &
   KILLER_PID=$!