[vlc-devel] [PATCH] simple_channel_mixer: Don't include the common AM_LDFLAGS for the arm_neon static library

Thomas Guillem thomas at gllm.fr
Mon Jul 1 15:04:53 CEST 2019


LGTM

On Sun, Jun 30, 2019, at 23:16, Martin Storsjö wrote:
> This avoids including module.rc.o in the helper static library when
> building for windows, avoiding a duplicate module.rc.o for the
> real simple_channel_mixer plugin.
> 
> Recent lld versions error out if two resource object files are passed
> to the same link. (If GNU ld is presented with two resource object
> files, it tries to merge the resources. In this case, as both object
> files contain the same resource, they would conflict, and GNU ld would
> print an error message, but still complete the link successfully.)
> ---
>  modules/audio_filter/Makefile.am | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/modules/audio_filter/Makefile.am 
> b/modules/audio_filter/Makefile.am
> index fde73b968d..309074c75b 100644
> --- a/modules/audio_filter/Makefile.am
> +++ b/modules/audio_filter/Makefile.am
> @@ -67,6 +67,9 @@ libsimple_channel_mixer_plugin_la_LIBADD =
>  if HAVE_NEON
>  EXTRA_LTLIBRARIES += libsimple_channel_mixer_plugin_arm_neon.la
>  libsimple_channel_mixer_plugin_arm_neon_la_SOURCES = 
> arm_neon/simple_channel_mixer.S
> +# Intentionally leaving out AM_LDFLAGS from this one; it's not meant 
> to be
> +# built like a plugin.
> +libsimple_channel_mixer_plugin_arm_neon_la_LDFLAGS = -static
>  
>  libsimple_channel_mixer_plugin_la_LIBADD += 
> libsimple_channel_mixer_plugin_arm_neon.la
>  libsimple_channel_mixer_plugin_la_CFLAGS += -DCAN_COMPILE_NEON
> -- 
> 2.17.1
> 
> _______________________________________________
> 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