This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch 1/2] Provide $ddir substitution for --with-auto-load-safe-path


On Wed, 09 May 2012 22:08:14 +0200, Joel Brobecker wrote:
> It's a nice addition to have the auto-load safe-path relocatable...
> 
> > +--with-auto-load-safe-path
> > +  Configure default value for the 'set auto-load safe-path' setting
> > +  above.  It defaults to '$ddir/auto-load', $ddir representing the value
> > +  of configure option --with-gdb-datadir.
> 
> I don't think that this is entirely accurate. $ddir represents the
> GDB data dir, whose value can easily be different from the value passed
> via --with-gdb-datadir.  I would refer to the value of "set
> data-directory" instead.

I have changed it in the patchset as you are right it is more correct.

But in reality user will use either configure option --with-gdb-datadir or the
command-line option -data-directory which is why I wrote it that way.
Until recently I did not remember how is 'show data-directory' exactly called.
But explaining there all of 'show data-directory', --with-gdb-datadir and
-data-directory is probably out of the scope of the NEWS file.

But I agree it is formally correct now.


Thanks,
Jan


NEWS:
-  above.  It defaults to '$ddir/auto-load', $ddir representing the value
-  of configure option --with-gdb-datadir.
+  above.  It defaults to '$ddir/auto-load', $ddir representing GDB's
+  data directory (available via show data-directory).

gdb.texinfo:
-Any used string @file{$ddir} will get replaced by @var{data-directory} which is
-determined at @value{GDBN} startup (@pxref{Data Files}).  @file{$ddir} must be
+Any used string @file{$ddir} will get replaced by @var{data-directory}, where
+@var{data-directory} is @value{GDBN}'s data directory (available via
+@code{show data-directory}, @pxref{Data Files}).  @file{$ddir} must be


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]