[webkit-reviews] review denied: [Bug 208330] [Win] Fix AppleWin build : [Attachment 391893] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 27 11:55:58 PST 2020


Don Olmstead <don.olmstead at sony.com> has denied Per Arne Vollan
<pvollan at apple.com>'s request for review:
Bug 208330: [Win] Fix AppleWin build
https://bugs.webkit.org/show_bug.cgi?id=208330

Attachment 391893: Patch

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




--- Comment #3 from Don Olmstead <don.olmstead at sony.com> ---
Comment on attachment 391893
  --> https://bugs.webkit.org/attachment.cgi?id=391893
Patch

This is wrong. You should be setting this to where the headers actually are.

I'm guessing you should move the setting of these values into the target files.
So like

set(WTF_FRAMEWORK_HEADERS_DIR "${CMAKE_BINARY_DIR}/../include/private/WTF")

Also https://bugs.webkit.org/show_bug.cgi?id=205085 still needs to land


More information about the webkit-reviews mailing list