[Buildroot] [PATCH v6 09/10] libopenmax: Add libopenmax virtual package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Thu May 16 14:22:16 UTC 2013


Dear Spenser Gilliland,

On Thu, 16 May 2013 09:18:38 -0500, Spenser Gilliland wrote:

> > but ok, that's certainly a different work. This current patch series is
> > already long enough.
> 
> I agree that a change like you suggested would be better but it's
> probably best left for another patch series due to the required
> changes being quite large.

Agreed, I'm fine with getting this part of your patch as is, and later
on, do some followup improvements. Especially since all those packages
and options have no effect on the configurations people may be
creating, some it doesn't cause any compatibility issue if we move
things around or rename those hidden kconfig options.

> >> +ifeq ($(BR2_PACKAGE_BELLAGIO),y)
> >> +LIBOPENMAX_DEPENDENCIES += bellagio
> >> +endif
> >
> > Unless I missed something, there is no bellagio package in Buildroot
> > for now, so this chunk should only be added once bellagio is added.
> 
> It's in package/multimedia/bellagio.

Argh, my bad. Stupid packages in subdirectories. I didn't remember
having bellagio, so I checked by doing package/bell<tab><tab>, and
nothing showed up, making me -incorrectly- conclude that this package
didn't exist.

Sorry for the noise on this one.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com


More information about the buildroot mailing list