[Buildroot] [PATCH] package/cegui: disable Python SWIG extension modules when OpenGL renderer is disabled

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sun Jul 18 20:27:36 UTC 2021


Hello Bartosz,

On Tue, 15 Jun 2021 21:45:36 +0200
Bartosz Bilas <b.bilas at grinn-global.com> wrote:

> Those modules need OpenGL renderer so disable them when it's missing.
> 
> Fixes:
>  - http://autobuild.buildroot.net/results/592/592fcffb226c8b44ada5fa3cdb8ab15c627f29dc/
>  - http://autobuild.buildroot.net/results/9ad/9ad768b7e2a3c59833e03986c245f2f4b5d25dcd/
> 
>   and more...
> 
> Signed-off-by: Bartosz Bilas <b.bilas at grinn-global.com>
> ---
>  package/cegui/cegui.mk | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/package/cegui/cegui.mk b/package/cegui/cegui.mk
> index 747ebbb953..c5ff45818e 100644
> --- a/package/cegui/cegui.mk
> +++ b/package/cegui/cegui.mk
> @@ -38,7 +38,8 @@ CEGUI_CONF_OPTS += -DCEGUI_USE_EPOXY=ON -DCEGUI_USE_GLEW=OFF \
>  else
>  CEGUI_CONF_OPTS += -DCEGUI_USE_EPOXY=OFF -DCEGUI_USE_GLEW=OFF \
>  	-DCEGUI_BUILD_RENDERER_OPENGL=OFF \
> -	-DCEGUI_BUILD_RENDERER_OPENGL3=OFF
> +	-DCEGUI_BUILD_RENDERER_OPENGL3=OFF \
> +	-DCEGUI_BUILD_PYTHON_MODULES_SWIG=OFF

Thanks for the patch, but shouldn't we set
-DCEGUI_BUILD_PYTHON_MODULES_SWIG=OFF unconditionally? Indeed, there is
nothing in cegui.mk that ensures that Python/SWIG is available prior to
the cegui build, i.e there is nothing in place to optionally build this
Python SWIG extension.

So if nobody uses it, we should rather disable it unconditionally. Do
you agree ?

Thanks a lot,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the buildroot mailing list