From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15845 invoked by alias); 24 Mar 2012 19:11:07 -0000 Received: (qmail 15835 invoked by uid 22791); 24 Mar 2012 19:11:05 -0000 X-SWARE-Spam-Status: No, hits=-1.1 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout21.012.net.il (HELO mtaout21.012.net.il) (80.179.55.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sat, 24 Mar 2012 19:10:51 +0000 Received: from conversion-daemon.a-mtaout21.012.net.il by a-mtaout21.012.net.il (HyperSendmail v2007.08) id <0M1E00K00L4GSY00@a-mtaout21.012.net.il> for gdb-patches@sourceware.org; Sat, 24 Mar 2012 21:10:50 +0200 (IST) Received: from HOME-C4E4A596F7 ([84.229.240.24]) by a-mtaout21.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0M1E00KBJLA0P480@a-mtaout21.012.net.il>; Sat, 24 Mar 2012 21:10:49 +0200 (IST) Date: Sat, 24 Mar 2012 19:11:00 -0000 From: Eli Zaretskii Subject: Re: [patch 2/6] set auto-load * main part In-reply-to: <20120324183747.GC26662@host2.jankratochvil.net> To: Jan Kratochvil Cc: gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <837gy9u85i.fsf@gnu.org> References: <20120324183747.GC26662@host2.jankratochvil.net> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-03/txt/msg00834.txt.bz2 > Date: Sat, 24 Mar 2012 19:37:47 +0100 > From: Jan Kratochvil > > --- a/gdb/NEWS > +++ b/gdb/NEWS This part is OK. > +Enable or disable auto-loading canned sequences of commands scripts."), _("\ > +Show enablement of auto-loading canned sequences of commands scripts."), _("\ Ouch, "enablement"... How about "state" instead? > +@item set auto-load local-gdbinit [yes|no] > +Enable or disable the auto-loading of > +@xref{Sequences, Canned Sequences of Commands}, found in init file in the > +current directory. This clever way of cross-references doesn't work. First @xref produces a capitalized "See", which is inappropriate in the middle of a sentence. But even if you replace it with @ref, there's still that "*note" that will look awkward. And finally, the "Sequences" part is displayed by Info readers, unlike HTML where the corresponding part of a reference would not be displayed. So I suggest this instead: Enable or disable the auto-loading of canned sequences of commands (@pxref{Sequences}) found in init file in the current directory. > +@cindex print commands from current directory init file have been auto-loaded This index entry is way too long. How about @cindex report auto-loading from current directory > @smallexample > -$ gdb -iex "set auto-load-scripts off" myprogram > +$ gdb -iex "set auto-load python-scripts" myprogram Did you really mean to omit the "off" part? > +@item set auto-load no > +Global disable of any auto-loading sub-commands of @value{GDBN}. You may want ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Globally disable any auto-loading ... > +Similar to the @ref{objfile-gdb.py file} file @value{GDBN} also tries to load > +an @file{@var{objfile}-gdb.rc} file containing > +@xref{Sequences, Canned Sequences of Commands}, as long as > +@samp{set auto-load gdb-scripts} is set to @samp{yes}. This @xref needs the same surgery as I show above. OK with those changes.