This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [doc ping] [patch+doc 2/2] auto-load: Fix default /usr/lib/debug/ loading
> Date: Sun, 20 May 2012 15:12:23 +0200
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> Cc: Eli Zaretskii <eliz@gnu.org>
>
> if it was forgotten, one doc review, please.
It looks like I simply missed it, sorry. Perhaps I though that this
was a trivial change wrt to the previous version I reviewed.
Anyway ...
> --- a/gdb/NEWS
> +++ b/gdb/NEWS
> @@ -201,8 +201,10 @@ show debug auto-load
>
> --with-auto-load-dir
> Configure default value for the 'set auto-load scripts-directory'
> - setting above. It defaults to '$datadir/auto-load', $datadir
> - representing GDB's data directory (available via show data-directory).
> + setting above. It defaults to '$debugdir:$datadir/auto-load',
> + $debugdir representing global debugging info directories (available
> + via 'show debug-file-directory') and $datadir representing GDB's data
> + directory (available via 'show data-directory').
OK for this part.
> +to (@pxref{with-auto-load-dir}). @file{$debugdir} and @file{$datadir}
> +substituation applies the same as for @xref{set auto-load scripts-directory}.
^^^^^^^^^^^^^ ^^^^^
A typo (yes, it was in the previous text as well). And please use
"@ref", not "@xref" (the latter is appropriate only at the beginning
of a sentence).
> +Any used string @file{$debugdir} will get replaced by @var{debug-file-directory}
> +value (@pxref{Separate Debug Files}) and any used string @file{$datadir} will
> +get replaced by @var{data-directory}
"Any reference to @file{$debugdir} will get replaced by
@var{debug-file-directory} ..."
and the same for the other variable.
OK with those changes.
Thanks.