From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1621 invoked by alias); 16 May 2002 02:32:41 -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 1610 invoked from network); 16 May 2002 02:32:39 -0000 Received: from unknown (HELO nevyn.them.org) (128.2.145.6) by sources.redhat.com with SMTP; 16 May 2002 02:32:39 -0000 Received: from drow by nevyn.them.org with local (Exim 3.35 #1 (Debian)) id 178B4P-0002O3-00; Wed, 15 May 2002 22:32:45 -0400 Date: Wed, 15 May 2002 19:32:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sources.redhat.com, mbp@samba.org Subject: Re: [patch] gdbserver usage message improvement Message-ID: <20020516023245.GA8942@nevyn.them.org> Mail-Followup-To: gdb-patches@sources.redhat.com, mbp@samba.org References: <20020516022026.GC26044@samba.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020516022026.GC26044@samba.org> User-Agent: Mutt/1.5.1i X-SW-Source: 2002-05/txt/msg00625.txt.bz2 On Thu, May 16, 2002 at 12:20:27PM +1000, Martin Pool wrote: > [I previously sent this to bug-gdb, but it looks like the newsgroup is > just full of spam, so I thought I'd resend it here.] > > This small patch brings gdbserver's usage message up to date with the > manual, so that it indicates it can support TCP as well as serial > connections. I remember being puzzled by this in the past, so I think > it would make gdb more friendly to have the message correct. > > As far as I can see TCP is supported on all platforms where gdbserver > builds. > > Thanks, > -- > Martin > (please cc me on replies) Thanks for the patch, I've committed it (after removing an extra pair of braces). FYI, we'd appreciate it if you could include ChangeLog entries with patches; saves me having to make one up myself :) 2002-05-15 Daniel Jacobowitz From Martin Pool : * gdbserver/server.c (gdbserver_usage): New function. (main): Call it. > > > cd /home/mbp/work/gdb/gdb-5.2/gdb/gdbserver/ > diff -u -p /home/mbp/work/gdb/gdb-5.2/gdb/gdbserver/server.c.\~1\~ /home/mbp/work/gdb/gdb-5.2/gdb/gdbserver/server.c > --- /home/mbp/work/gdb/gdb-5.2/gdb/gdbserver/server.c.~1~ Sat Mar 30 06:03:34 2002 > +++ /home/mbp/work/gdb/gdb-5.2/gdb/gdbserver/server.c Tue May 14 16:03:47 2002 > @@ -56,6 +56,16 @@ attach_inferior (int pid, char *statuspt > > extern int remote_debug; > > +static void > +gdbserver_usage (void) > +{ > + error ("Usage:\tgdbserver COMM PROG [ARGS ...]\n" > + "\tgdbserver COMM --attach PID\n" > + "\n" > + "COMM may either be a tty device (for serial debugging), or \n" > + "HOST:PORT to listen for a TCP connection.\n"); > +} > + > int > main (int argc, char *argv[]) > { > @@ -89,8 +99,9 @@ main (int argc, char *argv[]) > } > > if (argc < 3 || bad_attach) > - error ("Usage:\tgdbserver tty prog [args ...]\n" > - "\tgdbserver tty --attach pid"); > + { > + gdbserver_usage(); > + } > > initialize_low (); > > > Diff finished at Tue May 14 16:04:33 > -- Daniel Jacobowitz Carnegie Mellon University MontaVista Software Debian GNU/Linux Developer