Summary: | Build fails with GSTREAMER_GL when both desktop GL and GLES2 are enabled in gst-plugins-bad | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Olivier Blin <olivier.blin> | ||||||||
Component: | WebKitGTK | Assignee: | Miguel Gomez <magomez> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bugs-noreply, cgarcia, commit-queue, eocanha, magomez, mcatanzaro, pnormand, yoon, zan | ||||||||
Priority: | P2 | ||||||||||
Version: | Other | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Olivier Blin
2016-07-08 09:21:14 PDT
For reference, my build command is: ./Tools/Scripts/build-webkit --gtk --cmakeargs="-DUSE_GSTREAMER_GL=ON" I think Yoon had a similar issue, which was solved by changing the #include order of some files, IIRC. Created attachment 284017 [details]
Patch
Comment on attachment 284017 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=284017&action=review > Source/WebCore/ChangeLog:11 > + OpenGLShims.h redefines GL functions. How is OpenGLShims.h pulled in? Not sure I like the #include move to the Base.h header, we try to keep our headers free of #includes as much as possible. (In reply to comment #4) > Comment on attachment 284017 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=284017&action=review > > > Source/WebCore/ChangeLog:11 > > + OpenGLShims.h redefines GL functions. > > How is OpenGLShims.h pulled in? > > Not sure I like the #include move to the Base.h header, we try to keep our > headers free of #includes as much as possible. I don't like it either, it would be easy to forget and break. Here is the include flow: MediaPlayerPrivateGStreamerBase.cpp MediaPlayerPrivateGStreamerBase.h TextureMapperGL.h (for TextureMapperGL::Flags m_textureMapperRotationFlag) GraphicsContext3D.h ANGLEWebKitBridge.h OpenGLShims.h And the enum can't be forward-declared. Maybe we should refactor the rotation flag code so that a member variable isn't needed? Comment on attachment 284017 [details]
Patch
(Removing from request queue based on comments above.)
Created attachment 289344 [details]
Patch
Comment on attachment 289344 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=289344&action=review > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:117 > + return 0; ASSERT_NOT_REACHED() ? Created attachment 289348 [details]
Patch
(In reply to comment #10) > Created attachment 289348 [details] > Patch I can confirm this fixes the build, thanks! Comment on attachment 289348 [details] Patch Clearing flags on attachment: 289348 Committed r206202: <http://trac.webkit.org/changeset/206202> All reviewed patches have been landed. Closing bug. |