[vlc-devel] [PATCH 3/5] contrib: add an option to generate PDB files
robux4 at ycbcr.xyz
Thu May 16 10:36:06 CEST 2019
On 2019-05-16 10:27, Martin Storsjö wrote:
> On Thu, 16 May 2019, Steve Lhomme wrote:
>> On 2019-05-16 10:16, Martin Storsjö wrote:
>>> On Thu, 16 May 2019, Steve Lhomme wrote:
>>>> Only supported by contribs using libtool
>>>> Only output for programs and shared libraries (EXE and DLL), in .a
>>>> archives the
>>>> symbols are in the the each .o files. They are aggregated when a
>>>> .dll or .exe
>>>> is generated with them.
>>> As contribs for windows mainly build static archives (the built
>>> target executables aren't really used, and normally no DLLs are
>>> built), is there any actual need to patch libtool at all, and try to
>>> generate PDB files for these libs? For building just static libraries
>>> that contribute the right kind of debug info, wouldn't just the
>>> -gcodeview flag be enough?
>> Technically yes, static libraries don't have a PDB. But there are a
>> few .exe that we build. It may be useful to debug them if needed. IMO
>> it doesn't do any harm and can be useful.
> Well, no, it doesn't do any harm, no.
What kind of harm does it do ?
>It's just the extra weight of the
> libtool patch in 1/5 plus most lines in this patch.
> If the explicit intent is to have debuggable exes for the contrib
> builds, then I guess that's fine. I'd appreciate if that aspect is made
> clear, that the whole libtool patch isn't necessary for anything else
> than that.
I think I'll split the patch in two to make things clearer. I don't
really care about the --generate-pdb part.
More information about the vlc-devel