Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
X-WM-Posted-At:	avacado.atomice.net; Sun, 30 Jun 02 13:50:20 +0100
Message-ID:	<009f01c22034$b153c2c0$0100a8c0@advent02>
From:	"Chris January" <chris AT atomice DOT net>
To:	<cygwin AT cygwin DOT com>
Subject:	dup tty error.
Date:	Sun, 30 Jun 2002 13:50:20 +0100
MIME-Version:	1.0
X-Priority:	3
X-MSMail-Priority:	Normal
X-MimeOLE:	Produced By Microsoft MimeOLE V6.00.2600.0000
Note-from-DJ:	This may be spam

------=_NextPart_000_009B_01C2203D.12DAFAB0
Content-Type: text/plain;
	charset="utf-8"
Content-Transfer-Encoding: 8bit

Normally i would expect to see this error, say, 1 in 2 times I click the
Cygwin icon. But yesterday I clicked it about 30 times and I keep getting
the same error.
Error follows:
      3 [main] bash 776 fhandler_base::dup: dup(/dev/tty) failed, handle B,
Win32 error 87
readline: warning: rl_prep_terminal: cannot get terminal settings←]0;~
←[32mchris AT ADVENT02 ←[33m~←[0m
See this thread for more details:
http://sources.redhat.com/ml/cygwin/2002-05/msg01415.html

Attached is a patch which fixes the bug, by giving the console window title
more time (up to 1 second) to change. FindWindow is called in a loop, so for
the normal execution path there is no extra overhead.

Chris

---
2002-06-30  Christopher January <chris AT atomice DOT net>

 * tty.cc (tty_list::allocate_tty): retry FindWindow if it fails.


------=_NextPart_000_009B_01C2203D.12DAFAB0
Content-Type: text/plain;
	name="Changelog.tty.txt"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
	filename="Changelog.tty.txt"

2002-06-30  Christopher January <chris AT atomice DOT net>

	* tty.cc (tty_list::allocate_tty): retry FindWindow if it fails.

------=_NextPart_000_009B_01C2203D.12DAFAB0
Content-Type: application/octet-stream;
	name="tty.cc.patch"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: attachment;
	filename="tty.cc.patch"

Index: tty.cc=0A=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=0A=
RCS file: /cvs/src/src/winsup/cygwin/tty.cc,v=0A=
retrieving revision 1.35=0A=
diff -u -3 -p -u -p -r1.35 tty.cc=0A=
--- tty.cc	5 Jun 2002 04:01:43 -0000	1.35=0A=
+++ tty.cc	30 Jun 2002 12:42:04 -0000=0A=
@@ -216,8 +216,11 @@ tty_list::allocate_tty (int with_console=0A=
 =0A=
       __small_sprintf (buf, "cygwin.find.console.%d", myself->pid);=0A=
       SetConsoleTitle (buf);=0A=
-      Sleep (40);=0A=
-      console =3D FindWindow (NULL, buf);=0A=
+      for (int times =3D 0; times < 25 && console =3D=3D NULL; times++)=0A=
+	    {=0A=
+		  Sleep (40);=0A=
+          console =3D FindWindow (NULL, buf);=0A=
+	    }=0A=
       SetConsoleTitle (oldtitle);=0A=
       Sleep (40);=0A=
       ReleaseMutex (title_mutex);=0A=


------=_NextPart_000_009B_01C2203D.12DAFAB0
Content-Type: text/plain; charset=us-ascii

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/
------=_NextPart_000_009B_01C2203D.12DAFAB0--

- Raw text -