[fpc-devel] RTL StackTrace patch

[Tomas Hajny]

On Thu, January 17, 2008 10:59, Peter Vreman wrote:
>>>> I think it should by default look for program.dbg file , and then if
>>>> not
>>>> exists for paramstr(0). Is that enough ? If so,please explain how to
>>>> check if file exists , which function could be used here.
>>>
>>> No, as Peter described, you should look in the executable for a link to
>>> the file
>>> with debug info.
>>
>> Moreover, if the debug information is included within the executable
>> file
>> directly, there's probably no need to look for an external file, IMHO,
>> so
>> if the proper link isn't available and we would want to support external
>> files anyway, the preference should be probably the other way round
>> (first
>> paramstr(0), then paramstr(0) with extension changed to .dbg or
>> something
>> like that).
>
> I don't agree. We should not support looking implicitly for a .dbg without
> a gnu_debuglink
> section. Because there is no garauntee that the .dbg and the .exe will
> match. The gnu_debuglink is
> the only reliable way, because it uses a build-id or a checksum
> verification.

I don't think there's a contradiction - I wrote "if ... we would want to
support external files". I understand very well that we might not want to
support that for reasons you just mentioned.

Tomas