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

Martin Storsjö martin at martin.st
Sun Jun 30 23:16:10 CEST 2019


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



More information about the vlc-devel mailing list