[webkit-reviews] review denied: [Bug 202490] CMake-built WebKit.framework should launch XPC services successfully : [Attachment 380124] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 3 10:41:25 PDT 2019


Konstantin Tokarev <annulen at yandex.ru> has denied  review:
Bug 202490: CMake-built WebKit.framework should launch XPC services
successfully
https://bugs.webkit.org/show_bug.cgi?id=202490

Attachment 380124: Patch

https://bugs.webkit.org/attachment.cgi?id=380124&action=review




--- Comment #8 from Konstantin Tokarev <annulen at yandex.ru> ---
Comment on attachment 380124
  --> https://bugs.webkit.org/attachment.cgi?id=380124
Patch

I don't agree with this fix, for reasons mentioned above. Things specific to
Mac port should go to PlatformMac.cmake, not spread around. Also, if (APPLE)
condition will be true for WebKitGTK on macOS, which may have undesirable
consequences.


More information about the webkit-reviews mailing list