The WebKit2 plugin process currently links against libwebkit2gtk which itself links against GTK+ 3. Since most plugins link against GTK+ 2 we need the plugin process to link against GTK+ 2.
Created attachment 219061 [details] Patch
Created attachment 219298 [details] Patch fixing small dependency issue
Comment on attachment 219298 [details] Patch fixing small dependency issue View in context: https://bugs.webkit.org/attachment.cgi?id=219298&action=review Not for review yet? > Source/WebKit2/PlatformGTK.cmake:484 > + # FIXME: We should figure out a way to avoid compiling files that are common between the plugin > + # process and WebKit2 only once instead of recompiling them for the plugin process. Maybe build them into an intermediate library that is linked by both?
(In reply to comment #3) > (From update of attachment 219298 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=219298&action=review > > Not for review yet? Oops! webkit-patch was broken recently and I uploaded it manually. > > Source/WebKit2/PlatformGTK.cmake:484 > > + # FIXME: We should figure out a way to avoid compiling files that are common between the plugin > > + # process and WebKit2 only once instead of recompiling them for the plugin process. > > Maybe build them into an intermediate library that is linked by both? I think that's the best solution. It requires removing the files from the shared source list and then adding them to an intermediate library. It seemed a bit too much for this patch, so I hope it can be tackled later.
Comment on attachment 219298 [details] Patch fixing small dependency issue View in context: https://bugs.webkit.org/attachment.cgi?id=219298&action=review >>> Source/WebKit2/PlatformGTK.cmake:484 >>> + # process and WebKit2 only once instead of recompiling them for the plugin process. >> >> Maybe build them into an intermediate library that is linked by both? > > I think that's the best solution. It requires removing the files from the shared source list and then adding them to an intermediate library. It seemed a bit too much for this patch, so I hope it can be tackled later. Absolutely!
Committed r160728: <http://trac.webkit.org/changeset/160728>