[GTK][WPE] Guard libgbm code, make it a conditional dependency for GTK
Created attachment 454878 [details] Patch
Comment on attachment 454878 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=454878&action=review Great! LGTM! > Source/cmake/OptionsGTK.cmake:389 > + > + if (USE_ANGLE_WEBGL) > + find_package(GBM REQUIRED) > + find_package(LibDRM REQUIRED) > + SET_AND_EXPOSE_TO_BUILD(USE_LIBGBM TRUE) > + endif () I would add in the file a dependency, in case someone tries to force one and not the other, we are not planning to implement other option for ANGLE: WEBKIT_OPTION_DEPEND(USE_ANGLE_WEBGL USE_LIBGBM) > Source/cmake/OptionsWPE.cmake:285 > +SET_AND_EXPOSE_TO_BUILD(USE_LIBGBM TRUE) Ditto.
I missed the comment, I'll add them later.
Committed r291392 (248522@main): <https://commits.webkit.org/248522@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 454878 [details].
<rdar://problem/90415985>
Comment on attachment 454878 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=454878&action=review >> Source/cmake/OptionsGTK.cmake:389 >> + endif () > > I would add in the file a dependency, in case someone tries to force one and not the other, we are not planning to implement other option for ANGLE: > > WEBKIT_OPTION_DEPEND(USE_ANGLE_WEBGL USE_LIBGBM) You can't force USE_LIBGBM, it's not public or configurable.