From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-89236-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 29395 invoked by alias); 25 Mar 2012 21:17:39 -0000
Received: (qmail 29386 invoked by uid 22791); 25 Mar 2012 21:17:38 -0000
X-SWARE-Spam-Status: No, hits=-1.2 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; Sun, 25 Mar 2012 21:17:25 +0000
Received: from conversion-daemon.a-mtaout21.012.net.il by a-mtaout21.012.net.il (HyperSendmail v2007.08) id <0M1G00400LM8WL00@a-mtaout21.012.net.il> for gdb-patches@sourceware.org; Sun, 25 Mar 2012 23:17:06 +0200 (IST)
Received: from HOME-C4E4A596F7 ([84.229.240.24]) by a-mtaout21.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0M1G004YMLSHI9O0@a-mtaout21.012.net.il>; Sun, 25 Mar 2012 23:17:06 +0200 (IST)
Date: Sun, 25 Mar 2012 21:17:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [patch 4/6] set auto-load safe-path
In-reply-to: <20120325190702.GB9008@host2.jankratochvil.net>
To: Jan Kratochvil <jan.kratochvil@redhat.com>
Cc: gdb-patches@sourceware.org
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <83bonks7n0.fsf@gnu.org>
References: <20120324183946.GE26662@host2.jankratochvil.net> <834ntdu7sh.fsf@gnu.org> <20120325190702.GB9008@host2.jankratochvil.net>
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2012-03/txt/msg00858.txt.bz2

> Date: Sun, 25 Mar 2012 21:07:02 +0200
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> Cc: gdb-patches@sourceware.org
> 
> On Sat, 24 Mar 2012 20:18:38 +0100, Eli Zaretskii wrote:
> > > Date: Sat, 24 Mar 2012 19:39:46 +0100
> > > From: Jan Kratochvil <jan.kratochvil@redhat.com>
> > > 
> > > +set auto-load safe-path <dir1>[:<dir2>...]
> > > +show auto-load safe-path
> > > +  Sets a list of directories safe to hold auto-loaded files.
> > 
> >   Set a list of directories from which it is safe to auto-load files.
> 
> Used this one.
> 
> Changed also "Controls" to Control" in the other patch.
> 
> 
> > > +Set the directories safe to hold auto-loaded files."), _("\
> > > +Show the directories safe to hold auto-loaded files."), _("\
> > 
> > Same here.
> 
> I think here the texts used definite article "the", used that:
> 
> Set the list of directories from which it is safe to auto-load files."), _("\
> Show the list of directories from which it is safe to auto-load files."), _("\
> 
> 
> > This text should explain what happens if an auto-load file is found in
> > a directory not on this list.  Is it silently ignored? ignored with a
> > warning message? is the user prompted for permission to load it?
> > something else?
> 
> Reworked this section a bit.
> 
> There are intentionally no questions as it would be annoying to answer 'y'
> each time.  User should rather set it in the .gdbinit file or on commandline
> instead just once.

All are fine with me, thanks.