[vlc-devel] [PATCH 4/5] configure: add an option to generate PDB files when building

[Steve Lhomme]

On 2019-05-16 21:35, Martin Storsjö wrote:
> On Thu, 16 May 2019, Steve Lhomme wrote:
> 
>> This will only be enabled when building windows targets and will only 
>> work when
>> compiling with Clang.
>> ---
>> bin/Makefile.am   |  3 +++
>> configure.ac      | 13 +++++++++++++
>> lib/Makefile.am   |  3 +++
>> modules/common.am |  3 +++
>> src/Makefile.am   |  3 +++
>> 5 files changed, 25 insertions(+)
> 
> Thanks; this patchset is much clearer.
> 
> However, if we'd go with tweaking the lld option (which is progressing 
> fine upstream at https://reviews.llvm.org/D62004), we could get rid of 
> patch 1 (the libtool patch) altogether, and get rid of all the changes 
> to Makefile.am here, by just adding -Wl,-pdb= to the global LDFLAGS.

One of the use of the libtool feature is that it handles the .pdb 
cleaning with "make clean".

I'm also not sure about pathes to find the .PDB when built with WSL (I 
haven't tried it yet). So I prefer to put the PDB next to the DLL, in 
the .libs folder. And that's a libtool folder, it's unknown to the 
makefiles.