Bug 68727 - Simple EGL clients should not be compiled when configured with --disable-egl
Summary: Simple EGL clients should not be compiled when configured with --disable-egl
Status: VERIFIED FIXED
Alias: None
Product: Wayland
Classification: Unclassified
Component: weston (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Wayland bug list
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-29 20:28 UTC by U. Artie Eoff
Modified: 2013-09-24 22:40 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
configure output (8.16 KB, text/plain)
2013-08-29 20:28 UTC, U. Artie Eoff
Details

Description U. Artie Eoff 2013-08-29 20:28:22 UTC
Created attachment 84881 [details]
configure output

When Weston is configured with '--disable-egl' it still attempts to build the Simple EGL Clients (see attached configure output).  This results in a failed compile/link step during make:

  CCLD     weston-subsurfaces
/usr/bin/ld: weston_subsurfaces-subsurfaces.o: undefined reference to symbol 'eglBindAPI'
/usr/bin/ld: note: 'eglBindAPI' is defined in DSO /home/uartie/Work/wayland/install/lib/libEGL.so.1 so try adding it to the linker command line
/home/uartie/Work/wayland/install/lib/libEGL.so.1: could not read symbols: Invalid operation
collect2: error: ld returned 1 exit status
make[3]: *** [weston-subsurfaces] Error 1

----

weston (master) heads/master-0-ge1b655d
weston (1.2) heads/1.2-0-g237c1c8
Comment 1 Kristian Høgsberg 2013-09-04 06:02:58 UTC
commit edb6a3bd0a802eb3c9cb71e4d29588d69980c2b4
Author: Kristian Høgsberg <krh@bitplanet.net>
Date:   Tue Sep 3 22:59:17 2013 -0700

    configure.ac: Only enable enable_simple_egl_clients if enable_egl is yes
    
    This simplifies a few checks and makes sure we only build the
    subsurfaces client if EGL is enabled.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=68727
Comment 2 U. Artie Eoff 2013-09-04 14:17:33 UTC
(In reply to comment #1)
> commit edb6a3bd0a802eb3c9cb71e4d29588d69980c2b4
> Author: Kristian Høgsberg <krh@bitplanet.net>
> Date:   Tue Sep 3 22:59:17 2013 -0700
> 
>     configure.ac: Only enable enable_simple_egl_clients if enable_egl is yes
>     
>     This simplifies a few checks and makes sure we only build the
>     subsurfaces client if EGL is enabled.
>     
>     https://bugs.freedesktop.org/show_bug.cgi?id=68727

This fixes it on master.  But, it's still an issue on the 1.2 branch (current stable).  Is it worthwhile to cherry-pick to 1.2?
Comment 3 U. Artie Eoff 2013-09-05 16:16:00 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > commit edb6a3bd0a802eb3c9cb71e4d29588d69980c2b4
> > Author: Kristian Høgsberg <krh@bitplanet.net>
> > Date:   Tue Sep 3 22:59:17 2013 -0700
> > 
> >     configure.ac: Only enable enable_simple_egl_clients if enable_egl is yes
> >     
> >     This simplifies a few checks and makes sure we only build the
> >     subsurfaces client if EGL is enabled.
> >     
> >     https://bugs.freedesktop.org/show_bug.cgi?id=68727
> 
> This fixes it on master.  But, it's still an issue on the 1.2 branch
> (current stable).  Is it worthwhile to cherry-pick to 1.2?

This issue can be worked around by adding '--disable-simple-egl-clients' to configure, also.  Nonetheless, by itself, this patch is probably not critical enough to warrant another 1.2 minor release.  Thus, it's probably best to say it's a "wontfix" for 1.2.  Closing.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.