From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8079 invoked by alias); 31 Jul 2005 21:20:39 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 8060 invoked by uid 22791); 31 Jul 2005 21:20:34 -0000 Received: from eastrmmtao01.cox.net (HELO eastrmmtao01.cox.net) (68.230.240.38) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Sun, 31 Jul 2005 21:20:34 +0000 Received: from white ([68.9.64.121]) by eastrmmtao01.cox.net (InterMail vM.6.01.04.00 201-2131-118-20041027) with ESMTP id <20050731212004.NRUR12912.eastrmmtao01.cox.net@white>; Sun, 31 Jul 2005 17:20:04 -0400 Received: from bob by white with local (Exim 3.35 #1 (Debian)) id 1DzLEL-0006NW-00; Sun, 31 Jul 2005 17:20:21 -0400 Date: Sun, 31 Jul 2005 21:20:00 -0000 From: Bob Rossi To: gdb-patches@sources.redhat.com, Nick Roberts Subject: Re: MI testsuite to use PTY for inferior Message-ID: <20050731212021.GA24144@white> Mail-Followup-To: gdb-patches@sources.redhat.com, Nick Roberts References: <17131.5769.342629.658975@farnswood.snap.net.nz> <20050730173855.GA21401@white> <17131.64575.780190.163527@farnswood.snap.net.nz> <20050730230309.GA22547@white> <20050731012111.GB13808@nevyn.them.org> <20050731131653.GC22547@white> <20050731153051.GA28158@nevyn.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050731153051.GA28158@nevyn.them.org> User-Agent: Mutt/1.3.28i X-SW-Source: 2005-07/txt/msg00239.txt.bz2 On Sun, Jul 31, 2005 at 11:30:52AM -0400, Daniel Jacobowitz wrote: > On Sun, Jul 31, 2005 at 09:16:53AM -0400, Bob Rossi wrote: > > On Sat, Jul 30, 2005 at 09:21:11PM -0400, Daniel Jacobowitz wrote: > > > On Sat, Jul 30, 2005 at 07:03:09PM -0400, Bob Rossi wrote: > > > > However, since that's pretty ugly, I'll take your suggestion and always > > > > create and assign inferior_pty a value. Then I'll check it for the value > > > > of "true" before executing any code. > > > > > > I recommend doing something different. Make the argument a flag, i.e. > > > "mi_gdb_start use-tty". Or "no-tty" depending on what you want the > > > default to be. > > > > If it would be OK, I'd prefer to just have the TTY work with all MI > > tests, not making it optional. I'd like to repost the patch with all of > > the problems found already, and with that additional change. Is this OK? > > > > My theory is that no FE can/should use MI with out separating the inferior > > output via a pty. So, it's OK to test GDB under these assumptions. > > Well, first, let me ask you a question. What is the intended fate of > the old mechanism for interleaved output? The new TTY method has at > least two limitations: This is very interesting. This branches into 2 areas, the testsuite and FE's). As far as the testsuite is concerned, if we add this patch optional (currently the way it is), then everything is fine for target's that don't support any TTY mechanism. If we add the patch unconditionally, then I would say that the target's that don't support TTY's are unsupported by GDB/MI. What is GDB's stance on supporting target's via GDB/MI that can not support creating TTY's? Off the top of my head, I don't think it makes sense to support these targets. You can not write a reliable FE under theses circumstances. The inferior can spew out anything it chooses, including partial MI fragments (if inferior == GDB). > - As far as I know, native Win32 targets can't use PTYs: > http://world.std.com/~jmhart/critcom.htm#UNIX%20Pseudoterminal > So, they'll probably need something different. In this scenario, I think Cygwin is the answer. Or use GDB/MI with an inferior program that doesn't output anything to the terminal. > - Remote targets that provide output currently aren't redirected onto > the PTY; instead they'll appear interleaved, just like before. In this scenario, I'm guessing from the sound of it that GDB just hasn't added support for this yet. So it's a GDB bug, right? I could look into this if I had some direction. > Also, Andrew pretty specifically asked you to leave the mi2-* tests > alone for this change. Well, he definatly was against it at first, and then I thought maybe he was changing his mind, http://sources.redhat.com/ml/gdb-patches/2005-02/msg00110.html Thanks, Bob Rossi