[webkit-reviews] review granted: [Bug 196655] [CMake][WinCairo] Separate copied headers into different directories : [Attachment 366843] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 8 15:05:20 PDT 2019

Michael Catanzaro <mcatanzaro at igalia.com> has granted Don Olmstead
<don.olmstead at sony.com>'s request for review:
Bug 196655: [CMake][WinCairo] Separate copied headers into different

Attachment 366843: Patch


--- Comment #4 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Comment on attachment 366843
  --> https://bugs.webkit.org/attachment.cgi?id=366843

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

I'm OK with this, but I'm a little reluctant that none of our CMake experts
have reviewed it yet. Please wait one day before landing to give time for
others to comment.

> Source/WebCore/CMakeLists.txt:20

Well this is confusing. What is the difference between JSC public and private
framework headers, and why is the private include dir accessible outside JSC?

More information about the webkit-reviews mailing list