There has been build warning by INTERFACE_LINK_LIBRARIES. CMake Warning (dev) in Source/WebKit2/CMakeLists.txt: Policy CMP0022 is not set: INTERFACE_LINK_LIBRARIES defines the link interface. Run "cmake --help-policy CMP0022" for policy details. Use the cmake_policy command to set the policy and suppress this warning. Target "WebKit2" has an INTERFACE_LINK_LIBRARIES property which differs from its LINK_INTERFACE_LIBRARIES properties. INTERFACE_LINK_LIBRARIES:
Created attachment 237041 [details] Patch
Created attachment 237042 [details] Patch
r110024 set the INTERFACE_LINK_LIBRARIES manually because there were build break on major linux distro. However, it was two years ago. It looks we can use it on top level CMakeLists.txt again.
Created attachment 237193 [details] Patch
Anyone take a look this patch ?
Comment on attachment 237193 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=237193&action=review rs=me > Source/JavaScriptCore/ChangeLog:8 > + Set the LINK_INTERFACE_LIBRARIES target property on the tol level CMakeLists.txt. typo: tol -> top > Source/WebCore/ChangeLog:8 > + Set the LINK_INTERFACE_LIBRARIES target property on the top level CMakeLists.txt. ditto > Source/WebKit2/ChangeLog:8 > + Set the LINK_INTERFACE_LIBRARIES target property on the top level CMakeLists.txt. ditto > ChangeLog:8 > + Set the LINK_INTERFACE_LIBRARIES target property on the tol level CMakeLists.txt. ditto
There are only two "tol" typo, not four. :)
Committed r173155: <http://trac.webkit.org/changeset/173155>
(In reply to comment #7) > There are only two "tol" typo, not four. :) Thanks Ossy ;) Landed after fixing those nits.