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: [RFA] [1/2] auto-loading scripts from .debug_gdb_scripts section


On Wed, Apr 21, 2010 at 1:37 PM, Eli Zaretskii <eliz@gnu.org> wrote:
>> Date: Wed, 21 Apr 2010 12:23:16 -0700
>> From: Doug Evans <dje@google.com>
>>
>> I've applied the requested doc changes, except that Auto-loading is
>> still moved up one level, moved from node Python API to node Python.
>
> Thanks.
>
>> +If this file does not exist, and if the parameter
>> +@code{debug-file-directory} is set (@pxref{Separate Debug Files}),
>> +then @value{GDBN} will use for its each separated directory component
>> +@code{component} the file named @file{@code{component}/@var{real-name}}, where
>> +@var{real-name} is the object file's real name, as described above.
>
> In my review, I suggested a better wording for this sentence. ?Please
> use it.
>

Ah.  Missed that:

> If this file does not exist, and if the parameter
> @code{debug-file-directory} is set (@pxref{Separate Debug Files}),
> then @value{GDBN} will look for the file in all of the directories
> mentioned in the value of @code{debug-file-directory}.

I think the new wording still isn't sufficiently clear.
It doesn't say that each element in debug-file-directory is
concatenated with the full file name.  The user might think that only
the base name of the file is used.
[At least that is my understanding of how debug-file-directory would be used.]


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