208907 2020-03-11 01:06:47 -0700 [GTK] graphics/GLContext.cpp fails compiling with error: ‘initializeOpenGLShims’ was not declared in this scope; did you mean ‘initializeOpenGLShimsIfNeeded’? 2022-02-15 00:40:13 -0800 1 1 1 Unclassified WebKit WebKitGTK WebKit Local Build Unspecified Unspecified RESOLVED DUPLICATE 236593 https://bugs.webkit.org/show_bug.cgi?id=236593 P2 Normal --- 1 dpa-webkit aperez aperez berto bugs-noreply cgarcia ews-watchlist gustavo leio mcatanzaro oldest_to_newest 1628560 0 dpa-webkit 2020-03-11 01:06:47 -0700 Compiling WebKitGkt 2.28.0 fails with: make[2]: Entering directory '/src/gnome/webkitgtk-2.28.0/build' [ 72%] Building CXX object Source/WebCore/CMakeFiles/WebCore.dir/platform/graphics/GLContext.cpp.o /src/gnome/webkitgtk-2.28.0/Source/WebCore/platform/graphics/GLContext.cpp: In function ‘bool WebCore::initializeOpenGLShimsIfNeeded()’: /src/gnome/webkitgtk-2.28.0/Source/WebCore/platform/graphics/GLContext.cpp:66:19: error: ‘initializeOpenGLShims’ was not declared in this scope; did you mean ‘initializeOpenGLShimsIfNeeded’? 66 | success = initializeOpenGLShims(); | ^~~~~~~~~~~~~~~~~~~~~ | initializeOpenGLShimsIfNeeded /src/gnome/webkitgtk-2.28.0/Source/WebCore/platform/graphics/GLContext.cpp: In member function ‘unsigned int WebCore::GLContext::version()’: /src/gnome/webkitgtk-2.28.0/Source/WebCore/platform/graphics/GLContext.cpp:171:71: error: ‘::glGetString’ has not been declared; did you mean ‘res_getString’? 171 | String versionString = String(reinterpret_cast<const char*>(::glGetString(GL_VERSION))); | ^~~~~~~~~~~ | res_getString /src/gnome/webkitgtk-2.28.0/Source/WebCore/platform/graphics/GLContext.cpp:171:83: error: ‘GL_VERSION’ was not declared in this scope; did you mean ‘GCC_VERSION’? 171 | String versionString = String(reinterpret_cast<const char*>(::glGetString(GL_VERSION))); | ^~~~~~~~~~ | GCC_VERSION make[2]: *** [Source/WebCore/CMakeFiles/WebCore.dir/build.make:7769: Source/WebCore/CMakeFiles/WebCore.dir/platform/graphics/GLContext.cpp.o] Error 1 make[2]: Leaving directory '/src/gnome/webkitgtk-2.28.0/build' 1628565 1 dpa-webkit 2020-03-11 01:47:14 -0700 GlContext.cpp contains: #if USE(GLX) #include "GLContextGLX.h" #include "OpenGLShims.h" #endif static bool initializeOpenGLShimsIfNeeded() { #if USE(OPENGL_ES) || USE(LIBEPOXY) return true; #else static bool initialized = false; static bool success = true; if (!initialized) { success = initializeOpenGLShims(); initialized = true; } return success; #endif } So when GLX, OPENGL_ES and LIBEPOXY are not defined, which somehow happens to be the case on my system, compilation fails. My system does have libepoxy 1.5.3. 1650946 2 leio 2020-05-09 12:43:05 -0700 Source/WebCore/platform/graphics/GLContext.cpp is including OpenGLShims.h only if GLX is enabled, but then using the initializeOpenGLShims function in other cases as well. I think the preprocessor conditionals should match better. As-is, it seems to guarantee a fail for a -DENABLE_X11_TARGET=OFF -DENABLE_OPENGL=ON build that picks OpenGL. Basically initializeOpenGLShimsIfNeeded seems to think that a EGL+full OpenGL (instead of GLX) doesn't exist or happen, but libepoxy isn't used at all for a webkitgtk build (I guess it might be used in webkit-wpe or something) 1650949 3 leio 2020-05-09 12:45:02 -0700 while there, I also notice that initializeOpenGLShimsIfNeeded tracks initialized and success states, to not call it multiple times - but initializeOpenGLShims already (or by now) does exactly that as well, so that feels superfluous there in initializeOpenGLShimsIfNeeded 1650960 4 398939 leio 2020-05-09 13:46:52 -0700 Created attachment 398939 Patch to make non-X EGL full OpenGL builds succeed I'm not happy about this patch, as it doesn't adjust things to be similar in all the places, but enough to get the build issue I was having fixed. Looking into this, I found that various OpenGLShims.h usages are quite different from each other in their conditionals. So some places assume it's only needed for GLX, while others don't know anything about ANGLE being a thing yet, etc. I don't know the context and maze of compile conditionals enough to really know how to do this properly, so meanwhile I'm going to try using this attached patch in Gentoo. Note that this patch contains changes to another file issue as well, not just GLContext.cpp, as after fixing GLContext, the build reaches the next issue there. I'm going to also have it build various different build configurations in the background on a big aarch64 box; if I don't report back here, all of those succeed with this patch included. Otherwise I'll have to update the patch to make all the build combinations we try to support work, and will post it here as well for consideration or ideas. 1652213 5 mcatanzaro 2020-05-13 06:51:28 -0700 Please be careful when selecting Bugzilla components... if you don't select WebKitGTK, the bug is likely to be ignored. 1652218 6 mcatanzaro 2020-05-13 07:12:55 -0700 Trying to find when this broke... I'm not sure. That header was *added* to the file in r249909. Prior to that commit, it must have been included from some other header. The guards are clearly incorrect since the shims are initialized outside the USE(GLX) codepath, so your patch is an improvement at least. (In reply to Mart Raudsepp from comment #4) > Looking into this, I found that various OpenGLShims.h usages are quite > different from each other in their conditionals. So some places assume it's > only needed for GLX, while others don't know anything about ANGLE being a > thing yet, etc. Yeah I agree this seems to be pretty messed up. Needs to be investigated by graphics developers. 1667532 7 aperez 2020-06-30 04:26:52 -0700 (In reply to Mart Raudsepp from comment #2) > Source/WebCore/platform/graphics/GLContext.cpp is including OpenGLShims.h > only if GLX is enabled, but then using the initializeOpenGLShims function in > other cases as well. I think the preprocessor conditionals should match > better. > As-is, it seems to guarantee a fail for a -DENABLE_X11_TARGET=OFF > -DENABLE_OPENGL=ON build that picks OpenGL. The ENABLE_OPENGL option does not exist, I suppose the idea is to end up with USE_OPENGL=ON, which means setting ENABLE_GRAPHICS_CONTEXT_GL=ON. So yes, I can reproduce the issue by configuring a build with: % cmake -DPORT=GTK -DENABLE_X11_TARGET=OFF -DENABLE_GRAPHICS_CONTEXT_GL=ON > Basically initializeOpenGLShimsIfNeeded seems to think that a EGL+full > OpenGL (instead of GLX) doesn't exist or happen, but libepoxy isn't used at > all for a webkitgtk build (I guess it might be used in webkit-wpe or > something) The WPE port uses libepoxy, but the GTK port does not. 1667597 8 mcatanzaro 2020-06-30 09:15:01 -0700 (In reply to Adrian Perez from comment #7) > The WPE port uses libepoxy, but the GTK port does not. It's 2020, probably time to depend on libepoxy? 1739237 9 ews-watchlist 2021-03-14 09:34:38 -0700 Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API 1779326 10 mcatanzaro 2021-07-24 08:41:32 -0700 Hey Adrian, I think we need someone more familiar with OpenGL to unbreak the -DENABLE_X11_TARGET=OFF builds. Any chance you could help? (In reply to Michael Catanzaro from comment #8) > It's 2020, probably time to depend on libepoxy? libepoxy has been around for long enough that I think it's entirely reasonable to make it a hard dep. I assume that would make things a lot simpler. 1812554 11 aperez 2021-11-08 01:05:18 -0800 (In reply to Michael Catanzaro from comment #10) > Hey Adrian, I think we need someone more familiar with OpenGL to unbreak the > -DENABLE_X11_TARGET=OFF builds. Any chance you could help? > > (In reply to Michael Catanzaro from comment #8) > > It's 2020, probably time to depend on libepoxy? > > libepoxy has been around for long enough that I think it's entirely > reasonable to make it a hard dep. I assume that would make things a lot > simpler. I hadn't stepped in before because I didn't manage to reproduce the build issue, but recently this build issue came up with 2.32.x in one of the Buildroot autobuilders: http://autobuild.buildroot.net/results/2fbbf65a3dd8b6916fadb459af1b308f88b58554/ This means there we have now a build config file that can be used to trigger this build failure deterministically. I will give it a go and see if this is still a problem in 2.34.x and in trunk :) 1812560 12 leio 2021-11-08 02:39:41 -0800 In 2.34.x there's various issues with different build configurations. I'm applying https://gitweb.gentoo.org/repo/gentoo.git/tree/net-libs/webkit-gtk/files/2.34.1-opengl-without-X-fixes.patch but that's not enough for all combinations. Need to start with making sure non-unity builds work and then I think there's issues with non-X and/or non-GL builds in 2.34.x. 1812565 13 leio 2021-11-08 02:56:35 -0800 There are my downstream build issues of different failures from different build configurations, many can be tracked down from compilation failure, but I can help figuring out which build option triggers it too: https://bugs.gentoo.org/819384 https://bugs.gentoo.org/820437 https://bugs.gentoo.org/820440 https://bugs.gentoo.org/820443 Some are probably OpenGL related and on-topic here, others are other issues. I'm "leio" on Libera and GNOME IRC if that's better medium to work together on any of these. 1841606 14 aperez 2022-02-15 00:40:13 -0800 I believe the patch in bug #236593 should fix all the issues related to the OpenGLShims.h header. If you still find some remaining issue after it gets merged, please do reopen bug #236593 and let me know :) *** This bug has been marked as a duplicate of bug 236593 *** 398939 2020-05-09 13:46:52 -0700 2020-05-09 13:46:52 -0700 Patch to make non-X EGL full OpenGL builds succeed 2.28.2-opengl-without-X-fixes.patch text/plain 1656 leio RnJvbSBjNjdlZmEyYmJlMjA5NGI0MGI0ZTEwNGJiMjY0OTdjMmFmZjVjZTY4IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBNYXJ0IFJhdWRzZXBwIDxsZWlvQGdlbnRvby5vcmc+CkRhdGU6 IFNhdCwgOSBNYXkgMjAyMCAyMzoxMTo1MiArMDMwMApTdWJqZWN0OiBbUEFUQ0hdIENsZWFuIHVw IE9wZW5HTFNoaW1zLmggY29uZGl0aW9uYWxzIGluIGEgZmV3IHBsYWNlcyB0byBmaXgKIHNvbWUg YnVpbGQgY29uZmlndXJhdGlvbnMKCi0tLQogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhp Y3MvR0xDb250ZXh0LmNwcCAgICAgICAgICAgICB8IDUgKysrKy0KIFNvdXJjZS9XZWJLaXQvVUlQ cm9jZXNzL0FQSS9nbGliL1dlYktpdFByb3RvY29sSGFuZGxlci5jcHAgfCA1ICsrKystCiAyIGZp bGVzIGNoYW5nZWQsIDggaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQg YS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9HTENvbnRleHQuY3BwIGIvU291cmNl L1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvR0xDb250ZXh0LmNwcAppbmRleCBiMjE3OTg4Yjk5 MC4uMWJhMGViOGE0ODIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBo aWNzL0dMQ29udGV4dC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mv R0xDb250ZXh0LmNwcApAQCAtMjgsNiArMjgsOSBAQAogCiAjaWYgVVNFKEdMWCkKICNpbmNsdWRl ICJHTENvbnRleHRHTFguaCIKKyNlbmRpZgorCisjaWYgIVVTRShPUEVOR0xfRVMpICYmICFVU0Uo TElCRVBPWFkpICYmICFVU0UoQU5HTEUpCiAjaW5jbHVkZSAiT3BlbkdMU2hpbXMuaCIKICNlbmRp ZgogCkBAIC01Nyw3ICs2MCw3IEBAIGlubGluZSBUaHJlYWRHbG9iYWxHTENvbnRleHQqIGN1cnJl bnRDb250ZXh0KCkKIAogc3RhdGljIGJvb2wgaW5pdGlhbGl6ZU9wZW5HTFNoaW1zSWZOZWVkZWQo KQogewotI2lmIFVTRShPUEVOR0xfRVMpIHx8IFVTRShMSUJFUE9YWSkKKyNpZiBVU0UoT1BFTkdM X0VTKSB8fCBVU0UoTElCRVBPWFkpIHx8IFVTRShBTkdMRSkKICAgICByZXR1cm4gdHJ1ZTsKICNl bHNlCiAgICAgc3RhdGljIGJvb2wgaW5pdGlhbGl6ZWQgPSBmYWxzZTsKZGlmZiAtLWdpdCBhL1Nv dXJjZS9XZWJLaXQvVUlQcm9jZXNzL0FQSS9nbGliL1dlYktpdFByb3RvY29sSGFuZGxlci5jcHAg Yi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9BUEkvZ2xpYi9XZWJLaXRQcm90b2NvbEhhbmRsZXIu Y3BwCmluZGV4IDM1MTViZTQ1MmIyLi44MmNjZTcwZjE0YiAxMDA2NDQKLS0tIGEvU291cmNlL1dl YktpdC9VSVByb2Nlc3MvQVBJL2dsaWIvV2ViS2l0UHJvdG9jb2xIYW5kbGVyLmNwcAorKysgYi9T b3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9BUEkvZ2xpYi9XZWJLaXRQcm90b2NvbEhhbmRsZXIuY3Bw CkBAIC02NCw3ICs2NCwxMCBAQAogCiAjaWYgVVNFKEdMWCkKICNpbmNsdWRlIDxHTC9nbHguaD4K LSNpbmNsdWRlIDxXZWJDb3JlL09wZW5HTFNoaW1zLmg+CisjZW5kaWYKKworI2lmICFVU0UoT1BF TkdMX0VTKSAmJiAhVVNFKExJQkVQT1hZKSAmJiAhVVNFKEFOR0xFKQorI2luY2x1ZGUgIldlYkNv cmUvT3BlbkdMU2hpbXMuaCIKICNlbmRpZgogCiAjaWYgVVNFKEdTVFJFQU1FUikKLS0gCjIuMjAu MQoK