[vlc-devel] [PATCH 1/2] configure: allow toggling of -Wl, -z, defs and -no-undefined

[remi at remlab.net]

I don't understand what you are trying to suggest. In production builds, undefined symbols are usually forbidden either by design (PE) or by policy (ELF).

Le 20 juin 2017 09:30:54 GMT+03:00, Steve Lhomme <robux4 at gmail.com> a écrit :
>Can't it just be a flag that will only have effects on debug builds ?
>
>On Mon, Jun 19, 2017 at 5:56 PM, Rémi Denis-Courmont <remi at remlab.net>
>wrote:
>> Le maanantaina 19. kesäkuuta 2017, 17.02.49 EEST Hugo Beauzée-Luyssen
>a
>> écrit :
>>> I don't think the intention of this patch is to plainly disable
>>> no-undefined, but rather to allow someone to do it if required
>(which
>>> appears to be the case for ASAN on macOS, and TSAN on linux).
>>
>> Thing is, I really can´t believe that developpers will try a separate
>builds
>> at all times just for undefined symbols. If you give the option to
>disable it,
>> developpers *will* disable it at the first opportunity. In all
>likelihood, it
>> will end up in some random howto or build scripts too. Even more so
>if you
>> rely on it for sanitizers. Yet even more so if you require three
>builds, one
>> for address sanitization, one for thread sanitization and one for
>undefined
>> symbols.
>>
>> And then it will only be a matter of how often it breaks. That´s just
>how it
>> used to be before the option got enforced.
>>
>> --
>> 雷米‧德尼-库尔蒙
>> https://www.remlab.net/
>>
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170620/46aa5af0/attachment.html>