Re: [oe-core 14/20] mesa-dri: introduce MACHINE_DRI_MODULES

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Thu, 2011-10-13 at 13:30 +0200, Martin Jansa wrote:
> * not everybody needs i915, i965
> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/recipes-graphics/mesa/mesa-dri.inc     |    4 ++++
>  meta/recipes-graphics/mesa/mesa-dri_7.11.bb |    2 --
>  2 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-graphics/mesa/mesa-dri.inc b/meta/recipes-graphics/mesa/mesa-dri.inc
> index 603438e..be6905c 100644
> --- a/meta/recipes-graphics/mesa/mesa-dri.inc
> +++ b/meta/recipes-graphics/mesa/mesa-dri.inc
> @@ -6,6 +6,10 @@ DEFAULT_PREFERENCE = "-1"
>  
>  EXTRA_OECONF += "--with-driver=dri --disable-egl --disable-gallium --without-gallium-drivers"
>  
> +MACHINE_DRI_MODULES ?= ""
> +PACKAGE_ARCH = "${@['${MACHINE_ARCH}','${TUNE_PKGARCH}'][bb.data.getVar('MACHINE_DRI_MODULES',d,1) == '']}"
> +EXTRA_OECONF += "--with-dri-drivers=swrast,${MACHINE_DRI_MODULES}"
> +
>  python populate_packages_prepend() {
>  	import os.path

Whilst I understand the problem, I don't like this solution.
Particularly, it means that the meas-dri package needs to be marked as
machine specific which I don't like the idea of at all.

How about we do this on a per architecture basis?

Cheers,

Richard