[vlc-devel] [PATCH 2/3] contrib: Put the contrib in $host/$distro instead of just $host. This allows building for a same target with different distro type. (eg iPhone Simulator)

Pierre d'Herbemont pdherbemont at free.fr
Mon Sep 27 10:28:39 CEST 2010


2010/9/27 Rémi Denis-Courmont <remi at remlab.net>:
> On Sunday 26 September 2010, Pierre d'Herbemont wrote:
>> 2010/9/26 Rémi Denis-Courmont <remi at remlab.net>:
>> > Second, I claim your use case makes no sense.
>>
>> To build for the iPad, we need to have some host contrib (autoconf,
>> etc.) and then the target contrib. Then when trying to target the
>> simulator and the os, we need also separate contrib. I may be
>> hijacking the contrib to do this here. And we could figure out a
>> simpler/more solid way to solve this.
>
> First I don't see that use. autotools should be INSTALLED on the build system
> anyway, In fact, I don't see the point of having them in contribs in the first
> place. Even Cygwin (however crappy a build platform that is) has autotools.

Having host contrib makes the work flow easier for us. I actually
don't remember what more than a newer version of autotools we had.

But this could be reasonable to force the user to install a clean
autoconf, and the other contrib in PATH. And we could provide a build
script for that. I would be happy with this solution. Would you be
happy with this too? What else would you suggest?

> Second, that's not going to work cleanly with the current design of contribs
> anyway. You'd need to 'build' and 'install' autotools for each host platform,
> even though it would actually be targetted to the build platform.

Yes, that's what is being done by the iOS build script. Which is
anyway required.

So far, if I sum up:
- I am extending the contrib to make them work with various target.
The patch is rather thin.
- This tends to complexify the contrib system.
- You suggest to use an other system to install the host build
toolchain dependencies.

Am I mistaken?

Pierre.



More information about the vlc-devel mailing list