[OE-core] [PATCH v2] mesa-demos: Fix building demos which require GLU.
Moseley, Drew
drew_moseley at mentor.com
Thu Jul 16 16:38:42 UTC 2015
> On Jul 16, 2015, at 11:45 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
>
> On Thu, Jul 16, 2015 at 11:12:04AM -0400, Drew Moseley wrote:
>> The existing test for GLU support is backwards. Reverse the
>> sense of the conditional when built with "--enable-glu".
>>
>> Signed-off-by: Drew Moseley <drew_moseley at mentor.com <mailto:drew_moseley at mentor.com>>
>> ---
>> ...figure-Allow-to-disable-demos-which-require-GLEW-.patch | 14 ++++++++------
>> 1 file changed, 8 insertions(+), 6 deletions(-)
>>
>> diff --git a/meta/recipes-graphics/mesa/mesa-demos/0003-configure-Allow-to-disable-demos-which-require-GLEW-.patch b/meta/recipes-graphics/mesa/mesa-demos/0003-configure-Allow-to-disable-demos-which-require-GLEW-.patch
>> index 4b07193..b25f5ce 100644
>> --- a/meta/recipes-graphics/mesa/mesa-demos/0003-configure-Allow-to-disable-demos-which-require-GLEW-.patch
>> +++ b/meta/recipes-graphics/mesa/mesa-demos/0003-configure-Allow-to-disable-demos-which-require-GLEW-.patch
>> @@ -10,20 +10,21 @@ Subject: [PATCH 3/9] configure: Allow to disable demos which require GLEW or
>> Upstream-Status: Pending
>>
>> Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com <mailto:Martin.Jansa at gmail.com>>
>> +Signed-off-by: Drew Moseley <drew_moseley at mentor.com <mailto:drew_moseley at mentor.com>>
>> ---
>> - configure.ac | 49 ++++++++++++++++++++---------
>> - src/Makefile.am | 14 ++++++---
>> + configure.ac | 50 ++++++++++++++++++++---------
>> + src/Makefile.am | 18 ++++++++---
>> src/demos/Makefile.am | 73 ++++++++++++++++++++++++-------------------
>> src/egl/Makefile.am | 8 +++--
>> src/egl/opengles1/Makefile.am | 44 +++++++++++++++-----------
>> src/egl/opengles2/Makefile.am | 33 ++++++++++---------
>> - 6 files changed, 135 insertions(+), 86 deletions(-)
>> + 6 files changed, 140 insertions(+), 86 deletions(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index 9445424..bc4c8d1 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> -@@ -93,25 +93,44 @@ AC_EGREP_HEADER([glutInitContextProfile],
>> +@@ -89,25 +89,45 @@ AC_EGREP_HEADER([glutInitContextProfile],
>> [AC_DEFINE(HAVE_FREEGLUT)],
>> [])
>>
>> @@ -66,7 +67,7 @@ index 9445424..bc4c8d1 100644
>> -DEMO_CFLAGS="$DEMO_CFLAGS $GLU_CFLAGS"
>> -DEMO_LIBS="$DEMO_LIBS $GLU_LIBS"
>> +if test "x$enable_glu" = xyes; then
>> -+ PKG_CHECK_MODULES(GLU, [glu], [],
>> ++ PKG_CHECK_MODULES(GLU, [glu],
>
> Are you sure? It was like this in original PKG_CHECK_MODULES and I think
> it was correct. 4th parameter is used as fallback to find it manually
> when pkg-config doesn't find it, with your change it would use
> AC_CHECK_HEADER and ignore the values set returned by pkg-config.
Hi Martin,
I’m fairly certain this is needed however I’m not terribly familiar with autotools so it’s possible I missed something. I know that without this change a number of the demo apps I expect do not get built. It’s been a while since I looked but I seem to recall that the “geartrain” demo among others will not be available without this change.
From my reading of the PKG_CHECK_MODULES docs at https://autotools.io/pkgconfig/pkg_check_modules.html <https://autotools.io/pkgconfig/pkg_check_modules.html>, it seems the 3rd parameter is the action-if-found and my read of the logic here is that if glu is found it then verifies the presence of the header files and libs.
I’ll kick off a fresh build and dig around a bit just in case I missed something.
Regards,
Drew
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20150716/80eb350d/attachment-0002.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20150716/80eb350d/attachment-0002.sig>
More information about the Openembedded-core
mailing list