WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
121573
[GTK] Build fails against GTK2
https://bugs.webkit.org/show_bug.cgi?id=121573
Summary
[GTK] Build fails against GTK2
Dominique Leuenberger
Reported
2013-09-18 14:24:11 PDT
Attempting to build webkitgtk 2.1.92 fails when doing the GTK2 build: [ 1211s] Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp: In member function 'void WebKit::AcceleratedCompositingContext::initialize()': [ 1211s] Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp:88:36: error: 'GDK_IS_X11_DISPLAY' was not declared in this scope [ 1211s] if (!GDK_IS_X11_DISPLAY(display)) [ 1211s] ^ [ 1211s] make[1]: *** [Source/WebKit/gtk/WebCoreSupport/libwebkitgtk_1_0_la-AcceleratedCompositingContextGL.lo] Error 1 [ 1211s] make[1]: Leaving directory `/home/abuild/rpmbuild/BUILD/webkitgtk-2.1.92' [ 1211s] make: *** [all] Error 2 [ 1211s] error: Bad exit status from /var/tmp/rpm-tmp.g1Tb5F (%build)
Attachments
Add attachment
proposed patch, testcase, etc.
Dominique Leuenberger
Comment 1
2013-09-21 03:03:46 PDT
I applied this patch to my local tree, and this seems to do the trick: Index: webkitgtk-2.1.92/Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp =================================================================== --- webkitgtk-2.1.92.orig/Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp +++ webkitgtk-2.1.92/Source/WebKit/gtk/WebCoreSupport/AcceleratedCompositingContextGL.cpp @@ -30,6 +30,7 @@ #include "TextureMapperGL.h" #include "TextureMapperLayer.h" #include "webkitwebviewprivate.h" +#include "GtkVersioning.h" #include <wtf/CurrentTime.h> #if USE(OPENGL_ES_2)
Zan Dobersek
Comment 2
2013-09-22 23:42:38 PDT
Landed in
r156261
with proper attribution.
http://trac.webkit.org/changeset/156261
Added as a proposed merge for the 2.2 branch.
https://trac.webkit.org/wiki/WebKitGTK/2.2.x
Thanks for reporting this.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug