[vlc-devel] [RFC] [PATCH 0/2] IOMX support for Android
martin at martin.st
Wed Sep 21 22:47:04 CEST 2011
On Wed, 21 Sep 2011, Rémi Denis-Courmont wrote:
> Le mercredi 21 septembre 2011 15:49:31 Martin Storsjö, vous avez écrit :
>> - This creates a separate module "iomx", which builds from the
>> same source files as the "omxil" module. Is this ok, or would
>> it better to keep it named the same, but enabling this behaviour
>> only if some configure flag is specified? The separate module
>> has some ugly workaround to conditionally set a define when
>> building omxil.c for the iomx module.
> It's not (just) "ok", it's unavoidable. GNU/automake has a long-standing bug
> whereby the target is C++-linked whenever it has a C++ source, even when the
> C++ source is conditional and not currently used. If you had merged IOMX into
> OMX, the OMX plug-in would always link against libstdc++.
> See also the hacks to avoid C++ in the DTV access plug-in.
Oh, good point.
Yes, those parts of automake are a bit icky. On some other project, I set
libfoo_la_LINK = $(LINK) to force using the C linker.
More information about the vlc-devel