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

Steve Lhomme robux4 at gmail.com
Tue Jun 20 08:30:54 CEST 2017


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


More information about the vlc-devel mailing list