From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17240 invoked by alias); 11 Jul 2005 20:00:30 -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 17161 invoked by uid 22791); 11 Jul 2005 20:00:13 -0000 Received: from lakermmtao06.cox.net (HELO lakermmtao06.cox.net) (68.230.240.33) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Mon, 11 Jul 2005 20:00:13 +0000 Received: from white ([68.9.64.121]) by lakermmtao06.cox.net (InterMail vM.6.01.04.00 201-2131-118-20041027) with ESMTP id <20050711200008.CELY24173.lakermmtao06.cox.net@white>; Mon, 11 Jul 2005 16:00:08 -0400 Received: from bob by white with local (Exim 3.35 #1 (Debian)) id 1Ds4Rn-0000DO-00; Mon, 11 Jul 2005 16:00:11 -0400 Date: Mon, 11 Jul 2005 20:00:00 -0000 From: Bob Rossi To: Mark Kettenis Cc: gdb-patches@sources.redhat.com Subject: Re: mi tty commands Message-ID: <20050711200011.GA585@white> Mail-Followup-To: Mark Kettenis , gdb-patches@sources.redhat.com References: <20050602221644.GA21263@white> <20050613030732.GE9288@nevyn.them.org> <20050617210811.GA27959@white> <20050617211427.GA15201@nevyn.them.org> <20050618122921.GA30774@white> <20050703184325.GH13811@nevyn.them.org> <20050706145449.GA17788@white> <200507102108.j6AL8C9B031138@elgar.sibelius.xs4all.nl> <20050711155155.GA32323@white> <200507111840.j6BIeojH024633@elgar.sibelius.xs4all.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200507111840.j6BIeojH024633@elgar.sibelius.xs4all.nl> User-Agent: Mutt/1.3.28i X-SW-Source: 2005-07/txt/msg00096.txt.bz2 On Mon, Jul 11, 2005 at 08:40:50PM +0200, Mark Kettenis wrote: > Date: Mon, 11 Jul 2005 11:51:55 -0400 > From: Bob Rossi > > > Bob, please make sure there are no regressions from your patch before > > you check things in. I now get: > > > > FAIL: gdb.base/default.exp: tty > > FAIL: gdb.base/help.exp: help tty > > > > because the output of the tty command changed. In the past we had: > > > > (gdb) tty > > Argument required (terminal name for running target process). > > > > whereas now we have > > > > (gdb) tty > > Argument required (filename to set it to.). > > > > I don't really consider this an improvement... > > Does anyone know why the "tty" command gives the "default" message when > I have, > > /* add the filename of the terminal connected to inferior I/O */ > add_setshow_filename_cmd ("inferior-tty", class_run, > &inferior_io_terminal, _("\ > Set terminal for future runs of program being debugged."), _("\ > Show terminal for future runs of program being debugged."), _("\ > Usage: set inferior-tty /dev/pts/1"), NULL, NULL, &setlist, &showlist); > > add_com_alias ("tty", "set inferior-tty", class_alias, 0); > > I would expect the alias to take on the usage note from the command it > points to. Either that, or allow me to explicitly give it a usage note. > Any suggestions? > > But it does take the usage nore from the command it points to: > > (gdb) set inferior-tty > Argument required (filename to set it to.). Whoa, OK. I just realized that add_setshow_filename_cmd is borked. The "set_doc" doesn't work, but the "show_doc" does. Look at this command (which I had nothing to do with), add_setshow_filename_cmd ("remotelogfile", no_class, &serial_logfile, _("\ Set filename for remote session recording."), _("\ Show filename for remote session recording."), _("\ This file is used to record the remote session for future playback\n\ by gdbserver."), ... (gdb) set remotelogfile Argument required (filename to set it to.). (gdb) show remotelogfile Filename for remote session recording is "". I'll look into this. Bob Rossi