[vlc-devel] [RFC] [PATCH 0/2] IOMX support for Android

Martin Storsjö 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.

// Martin


More information about the vlc-devel mailing list