This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
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