Bug 135505 – [GTK] Be able to disable gtk2 dependency

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Milan Crha]

Please add a configure option to be able to disable gtk2 dependency. It's not good to mix gtk3 and gtk2 (I build webkitgtk for gtk3), and it's sometimes not needed (like for evolution). If you can enable it by default, but allow a configure option to disable it, then it'll help to build softer webkit (like under Win32, I know, I know, webkit2 doesn't work right now, but let's start with small steps). It's understood that disable of the gtk2 dependency removes some functionality, like the famous flash, but that's understood and fine.

[Carlos Garcia Campos]

Now that the default plugin process doesn't use GTK2, it's possible to build without gtk2 support. And of course GTK+2 plugins like flash would not work.

[Carlos Garcia Campos]

Created attachment 235969 [details]
Add an option to disable WebKitPluginProcess2

WebKitPluginProcess2 is the only reason we depend on GTK+2.

[Gustavo Noronha (kov)]

Comment on attachment 235969 [details]
Add an option to disable WebKitPluginProcess2

View in context: https://bugs.webkit.org/attachment.cgi?id=235969&action=review

> Source/WebKit2/PlatformGTK.cmake:-580
> -    list(APPEND PluginProcessGTK2_INCLUDE_DIRECTORIES
> -        "${WEBKIT2_DIR}/PluginProcess/unix"
> -    )
> -
> -    include_directories(${PluginProcessGTK2_INCLUDE_DIRECTORIES})

This path is already added on a more general basis, I imagine?

[Carlos Garcia Campos]

Comment on attachment 235969 [details]
Add an option to disable WebKitPluginProcess2

View in context: https://bugs.webkit.org/attachment.cgi?id=235969&action=review

>> Source/WebKit2/PlatformGTK.cmake:-580
>> -    include_directories(${PluginProcessGTK2_INCLUDE_DIRECTORIES})
> 
> This path is already added on a more general basis, I imagine?

Yes, see the previous chunk

[Carlos Garcia Campos]

Committed r172144: <http://trac.webkit.org/changeset/172144>