[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]

On Wed, Oct 6, 2010 at 9:41 AM, Jean-Paul Saman <jpsaman at gmail.com> wrote:
> On Fri, Oct 1, 2010 at 8:04 PM, Pierre d'Herbemont <pdherbemont at free.fr> wrote:
>> On Fri, Oct 1, 2010 at 4:11 PM, Jean-Baptiste Kempf <jb at videolan.org> wrote:
>>> On Sat, Sep 25, 2010 at 11:51:09PM +0200, Pierre d'Herbemont wrote :
>>>> -      CONTRIB_DIR="${topdir}/extras/contrib/hosts/${host}"
>>>> +      CONTRIB_DIR="${topdir}/extras/contrib/hosts/${host}/default_distro"
>>> I am not at ease with "default_distro", I have to say.
>>
>> Do you have a suggestion to throw in? This is referring to the $distro
>> in bootstrap, value that refers itself to Distribution/blah.mk.
>
> Maybe the contribs compile and build result should be done in a
> builddir instead of in the source tree. You then can have a contrib
> build for any combination you need. Eg:
>
> vlc-builddir-macosx
>   extras/contrib
>
> vlc-builddir-native
>   extras/contrib
>
> vlc-builddir-arm
>   extras/contrib
>
> What do you think?

This would be nice also. The only minor glitch is that we specify at
configure time the build directory. So far, this is nice if we can
specify a default build dir that is not "." with autoconf.

Also, this assumes that the host tools are not built by the contrib
system. Which is the case today. (In the case of the host (vs target)
contrib, this issue is completely orthogonal).

Pierre.