[Webkit-unassigned] [Bug 172862] [JSCOnly] Support building jsc.exe with dynamic JSC library on Windows

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=172862

Vladimir Vukicevic <vladimir at pobox.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vladimir at pobox.com

--- Comment #1 from Vladimir Vukicevic <vladimir at pobox.com> ---
Created attachment 315461

  --> https://bugs.webkit.org/attachment.cgi?id=315461&action=review

fix shared jsc shell builds on windows

The reason for this is that the JSCOnly builds expect a static WTF (but don't define WTF_LIBRARY_TYPE, fixed that) that's compiled in to JavaScriptCore, but they still make WTF as part of the cmake target library dependencies.  Additionally, the -DSTATICALLY_LINKED_WITH_WTF gets propagated down to the jsc shell, which is also not correct.

The library thing (jscore.dll needs to link with WTF, but it should *not* tell downstream dependants that they need to link with WTF.lib) I fixed by introducing a public/private target_link_libraries split in WEBKIT_FRAMEWORK, and adding WTF to the private libs for jscore as appropriate.

The STATICALLY_LINKED_WITH_WTF define was done the same way, by adding it to the PRIVATE defines for JavaScriptCore if WTF is static.

The static linking of WTF also causes some problems because we don't actually get the WTF includes or WTF's own dependant libs (like winmm).  I hacked around that in the shell CMakeLists, but I'm not very happy about it.

With this, both static and shared JSCOnly builds work on Windows.  I have not tested any other config or any other platform, though.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170714/5856740a/attachment.html>