After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 739348 - configure.ac: auto decision to include GL library fails
configure.ac: auto decision to include GL library fails
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.4.3
Other Linux
: Normal normal
: 1.4.4
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-10-29 11:16 UTC by Vincent Abriou
Modified: 2014-10-29 11:43 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
fix openGL detection (1.20 KB, patch)
2014-10-29 11:16 UTC, Vincent Abriou 		committed Details | Review

Description Vincent Abriou 2014-10-29 11:16:51 UTC 
Created attachment 289578 [details] [review]
fix openGL detection

The part of the configure.ac that consist to check if we can include both GL and GLES2 at the same time is failing.
Indeed, in the case NEED_GLES2=yes and NEED_OPENGL=auto, HAVE_OPENGL variable is updated whereas it should be HAVE_GL variable that has to be updated (HAVE_OPENGL variable is not used in the rest of the configure.ac).
Comment 1 Matthew Waters (ystreet00) 2014-10-29 11:43:15 UTC 
Thanks!

commit 9f69bcbfe5ccca672c2042beafda94d37e677148
Author: Vincent Abriou <vincent.abriou@st.com>
Date:   Wed Oct 29 10:09:35 2014 +0100

    configure.ac: auto decision to include GL library fails
    
    The part of the configure.ac that consist to check if we
    can include both GL and GLES2 at the same time is failing.
    Indeed, in the case NEED_GLES2=yes and NEED_OPENGL=auto,
    HAVE_OPENGL variable is updated whereas it should be HAVE_GL
    variable that has to be updated (HAVE_OPENGL variable is not
    used in the rest of the configure.ac).
    
    https://bugzilla.gnome.org/show_bug.cgi?id=739348
    
    Signed-off-by: Vincent Abriou <vincent.abriou@st.com>
    Reviewed-by: Benjamin GAIGNARD <benjamin.gaignard@linaro.org>

and 1.4 aad5b0d16646d708415120d4618385d9d70e450f