* [Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util
@ 2023-08-27 15:18 Romain Naour
2023-08-27 16:23 ` Julien Olivain
2023-09-02 15:52 ` Thomas Petazzoni via buildroot
0 siblings, 2 replies; 3+ messages in thread
From: Romain Naour @ 2023-08-27 15:18 UTC (permalink / raw)
To: buildroot; +Cc: Romain Naour, Julien Olivain
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@gmail.com>
Cc: Julien Olivain <ju.o@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@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@free.fr>
[Bernd: rebased for version 9.0.0]
Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
+[Romain: fix egl only builds in util]
+Signed-off-by: Romain Naour <romain.naour@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
--
2.41.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util
2023-08-27 15:18 [Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util Romain Naour
@ 2023-08-27 16:23 ` Julien Olivain
2023-09-02 15:52 ` Thomas Petazzoni via buildroot
1 sibling, 0 replies; 3+ messages in thread
From: Julien Olivain @ 2023-08-27 16:23 UTC (permalink / raw)
To: Romain Naour; +Cc: Romain Naour, buildroot
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@gmail.com>
> Cc: Julien Olivain <ju.o@free.fr>
Reviewed-by: Julien Olivain <ju.o@free.fr>
Tested-by: Julien Olivain <ju.o@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@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@free.fr>
> [Bernd: rebased for version 9.0.0]
> Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
> +[Romain: fix egl only builds in util]
> +Signed-off-by: Romain Naour <romain.naour@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.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util
2023-08-27 15:18 [Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util Romain Naour
2023-08-27 16:23 ` Julien Olivain
@ 2023-09-02 15:52 ` Thomas Petazzoni via buildroot
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-09-02 15:52 UTC (permalink / raw)
To: Romain Naour; +Cc: Julien Olivain, buildroot
On Sun, 27 Aug 2023 17:18:28 +0200
Romain Naour <romain.naour@gmail.com> 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@gmail.com>
> Cc: Julien Olivain <ju.o@free.fr>
> ---
> ...s-makes-opengl-an-optional-component.patch | 95 ++++++++++---------
> 1 file changed, 50 insertions(+), 45 deletions(-)
Applied to master, thanks. Any chance that we try to upstream this
patch to the mesa3d-demos project?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-09-02 15:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-27 15:18 [Buildroot] [PATCH] package/mesa3d-demos: update optional opengl patch due to new glu dependency in util Romain Naour
2023-08-27 16:23 ` Julien Olivain
2023-09-02 15:52 ` Thomas Petazzoni via buildroot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox