Review Request 115457: Allow compiling KJS on windows

Alexander Richardson arichardson.kde at googlemail.com
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/
-----------------------------------------------------------

(Updated Feb. 4, 2014, 7:26 p.m.)


Status
------

This change has been marked as submitted.


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/1281d4b7/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list