[Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util
Julien Olivain
ju.o at free.fr
Sun Aug 27 16:23:11 UTC 2023
Hi Romain,
Thank you for the patch! I reproduced the build issue (without this
patch). I also successfully tested that this patch fixes the issue.
On 27/08/2023 17:18, Romain Naour wrote:
> matrix-code from es2gears has been moved to util in version 9.0.0 [1]
> so now es2gears is linked with mesa3d-demo libutil (idep_util).
> But at the same time the dependency on glu (dep_glu) has been added
> to libutil [2]. dep_glu requires opengl (FULL_GL) to build.
>
> In order to keep the es2gears for egl only builds we have to
> split the libutil files list to build readtex.c and showbuffer.c
> only if opengl is available.
>
> Add back dep_glu since it's used in util's meson.
>
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/4936948236
> https://gitlab.com/buildroot.org/buildroot/-/jobs/4936948042
>
> [1]
> https://gitlab.freedesktop.org/mesa/demos/-/commit/c33e2f731c2eab86bbad7b6a028f8a90ea40a00c
> [2]
> https://gitlab.freedesktop.org/mesa/demos/-/commit/ef24aae22978d4d357340074ca9a1c04d9cf81e6
>
> Signed-off-by: Romain Naour <romain.naour at gmail.com>
> Cc: Julien Olivain <ju.o at free.fr>
Reviewed-by: Julien Olivain <ju.o at free.fr>
Tested-by: Julien Olivain <ju.o at free.fr>
> ---
> ...s-makes-opengl-an-optional-component.patch | 95 ++++++++++---------
> 1 file changed, 50 insertions(+), 45 deletions(-)
>
> diff --git
> a/package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch
> b/package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch
> index af929003db..72bf72719c 100644
> ---
> a/package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch
> +++
> b/package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch
> @@ -1,25 +1,44 @@
> -From 9ec9f7b4d0ef5e799ca4cc5103163e4630d184d8 Mon Sep 17 00:00:00 2001
> +From 5b45f98255b74589015d5cb0d8ca31d7169c9f34 Mon Sep 17 00:00:00 2001
> From: Julien Olivain <ju.o at free.fr>
> Date: Fri, 16 Sep 2022 22:30:59 +0200
> Subject: [PATCH] demos: makes opengl an optional component
>
> This patch is a port of [1] for the new meson build system.
>
> +matrix-code from es2gears has been moved to util in version 9.0.0 [2]
> +so now es2gears is linked with mesa3d-demo libutil (idep_util).
> +But at the same time the dependency on glu (dep_glu) has been added
> +to libutil [3]. dep_glu requires opengl (FULL_GL) to build.
> +
> +In order to keep the es2gears for egl only builds we have to
> +split the libutil files list to build readtex.c and showbuffer.c
> +only if opengl is available.
> +
> +Add back dep_glu since it's used in util's meson.
> +
> +Fixes:
> +https://gitlab.com/buildroot.org/buildroot/-/jobs/4936948236
> +https://gitlab.com/buildroot.org/buildroot/-/jobs/4936948042
> +
> [1].
>
> https://git.busybox.net/buildroot/tree/package/mesa3d-demos/0001-demos-makes-opengl-an-optional-component.patch?h=2022.08
> +[2]
> https://gitlab.freedesktop.org/mesa/demos/-/commit/c33e2f731c2eab86bbad7b6a028f8a90ea40a00c
> +[3]
> https://gitlab.freedesktop.org/mesa/demos/-/commit/ef24aae22978d4d357340074ca9a1c04d9cf81e6
>
> Signed-off-by: Julien Olivain <ju.o at free.fr>
> [Bernd: rebased for version 9.0.0]
> Signed-off-by: Bernd Kuhls <bernd at kuhls.net>
> +[Romain: fix egl only builds in util]
> +Signed-off-by: Romain Naour <romain.naour at gmail.com>
> ---
> - meson.build | 19 ++++++++++++-------
> - meson_options.txt | 1 +
> - src/egl/meson.build | 5 ++++-
> - src/meson.build | 4 +++-
> - 4 files changed, 20 insertions(+), 9 deletions(-)
> + meson.build | 5 ++++-
> + meson_options.txt | 1 +
> + src/egl/meson.build | 5 ++++-
> + src/util/meson.build | 12 +++++++++---
> + 4 files changed, 18 insertions(+), 5 deletions(-)
>
> diff --git a/meson.build b/meson.build
> -index e1968fab..a15699d6 100644
> +index 3e39b5a2..f27e1a18 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -39,7 +39,10 @@ add_project_arguments(
> @@ -34,34 +53,8 @@ index e1968fab..a15699d6 100644
>
> dep_epoll = dependency('epoll-shim', required : false)
> dep_gles1 = dependency('glesv1_cm', required : get_option('gles1'))
> -@@ -87,15 +87,17 @@
> -
> - dep_threads = dependency('threads')
> -
> --dep_glu = dependency('glu', required : false)
> --# GLU is part of OpenGL.Framework
> --if not dep_glu.found() and host_machine.system() != 'darwin'
> -- _glu_name = 'GLU'
> -- if host_machine.system() == 'windows'
> -- _glu_name = 'glu32'
> -+if dep_gl.found()
> -+ dep_glu = dependency('glu', required : false)
> -+ # GLU is part of OpenGL.Framework
> -+ if not dep_glu.found() and host_machine.system() != 'darwin'
> -+ _glu_name = 'GLU'
> -+ if host_machine.system() == 'windows'
> -+ _glu_name = 'glu32'
> -+ endif
> -+ dep_glu = cc.find_library(_glu_name, has_headers: 'GL/glu.h',
> -+ required : dep_x11.found())
> - endif
> -- dep_glu = cc.find_library(_glu_name, has_headers: 'GL/glu.h',
> -- required : dep_x11.found())
> - endif
> -
> - dep_glx = dependency('glx', required: false, disabler : true)
> diff --git a/meson_options.txt b/meson_options.txt
> -index ddadd2e3..f5405698 100644
> +index 832b916a..c954a710 100644
> --- a/meson_options.txt
> +++ b/meson_options.txt
> @@ -1,5 +1,6 @@
> @@ -87,21 +80,33 @@ index d1846921..42bf8ecd 100644
>
> if dep_gles1.found()
> subdir('opengles1')
> -diff --git a/src/meson.build b/src/meson.build
> -index 39cac78d..7607c6de 100644
> ---- a/src/meson.build
> -+++ b/src/meson.build
> -@@ -20,7 +20,9 @@
> +diff --git a/src/util/meson.build b/src/util/meson.build
> +index b4fe614f..4420f28a 100644
> +--- a/src/util/meson.build
> ++++ b/src/util/meson.build
> +@@ -22,13 +22,19 @@ inc_util = include_directories('.')
>
> - subdir('glad')
> + files_libutil = files(
> + 'glinfo_common.c',
> +- 'readtex.c',
> +- 'showbuffer.c',
> + 'trackball.c',
> + 'matrix.c',
> + )
>
> --subdir('util')
> +-_deps = [dep_glu, dep_m]
> ++_deps = [dep_m]
> +if dep_gl.found()
> -+ subdir('util')
> ++ files_libutil += files(
> ++ 'readtex.c',
> ++ 'showbuffer.c',
> ++ )
> ++ _deps += dep_glu
> +endif
> -
> ++
> if dep_glut.found()
> - subdir('demos')
> + files_libutil += files('shaderutil.c')
> + _deps += dep_glut
> --
> -2.37.3
> +2.41.0
Best regards,
Julien.
More information about the buildroot
mailing list