Review Request 115457: Allow compiling KJS on windows
Commit Hook
null at kde.org
Tue Feb 4 19:26:09 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115457/#review48968
-----------------------------------------------------------
This review has been submitted with commit ba28d6fe8cc89258c6de612c57d6b965e30623bc by Alex Richardson to branch master.
- Commit Hook
On Feb. 4, 2014, 3:54 p.m., Alexander Richardson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115457/
> -----------------------------------------------------------
>
> (Updated Feb. 4, 2014, 3:54 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Repository: kjs
>
>
> Description
> -------
>
> A few small commits, I only made one review request since it is only a small patch alltogether
>
>
> Use the correct define _WIN32 instead of WIN32
>
> This fixes the build with MSVC
>
> -----------------------------
>
> Fix MSVC linking of the test executables
>
> -------------------------------
>
> Fix includes on Windows
>
> timeapi.h is needed for timeGetTime() and xmath.h no longer exists with
> newer MSVC versions
>
> ------------------------------------------------------
>
> Fix the PLATFORM(), COMPILER() etc. macros on MSVC
>
> MSVC does not handle expanding defined() inside macros
>
> -------------------------------------------------
>
> Fix linkage on MSVC
>
> We have to define MAKE_KJS_LIB to get dllimport/dllexport right
>
> -------------------------------------------------------
>
> Get rid of KDEWIN_INCLUDES and KDEWIN_LIBRARIES
>
> These variables no longer exist
>
> ------------------------------------------------------------
>
> Use ecm_mark_nongui_executable to fix linking on Windows
>
>
> Diffs
> -----
>
> CMakeLists.txt 99b0c4edd1f641900527ea79b13dba230a1025e2
> tests/testkjs.cpp a00ed144b54896842beb9a6519d107fdff102319
> tests/CMakeLists.txt 71664cc7dd4df31ed56243440666217e1177619c
> src/wtf/Platform.h 39e55dac1ba69feee47b8609cf05b91541f9740b
> src/wtf/MathExtras.h 50fb006a259797b16bfce09b5e2d101cf127fb38
> src/kjs/object.h e2279cb026783dee1ffb2c737f6181de0f41464f
> src/kjs/function.h c09bd385ff2aa1dd48a80aa1032d26787514ce04
> src/kjs/CMakeLists.txt c1c8cef2bf1e44c8d762d6894310ce68ca05b2d7
> autotests/CMakeLists.txt 1268b8a6555fd36cd73f97beb6989b21056160f0
>
> Diff: https://git.reviewboard.kde.org/r/115457/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Alexander Richardson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140204/7cff5a19/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list