[CMake] Some macros need to be defined/undefined, rather than ON/OFF
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 11 May 2015 03:03:07 +0000 (03:03 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 11 May 2015 03:03:07 +0000 (03:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=144845

Patch by Philip Chimento <philip.chimento@gmail.com> on 2015-05-10
Reviewed by Martin Robinson.

* Source/cmake/OptionsGTK.cmake: Only define MOZ_X11 and XP_UNIX
if their corresponding WTF options are ON. The code in npapi.h
relies on these being undefined if they are to be switched off.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@184062 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Source/cmake/OptionsGTK.cmake

index 46850cb..eb7bc8a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2015-05-10  Philip Chimento  <philip.chimento@gmail.com>
+
+        [CMake] Some macros need to be defined/undefined, rather than ON/OFF
+        https://bugs.webkit.org/show_bug.cgi?id=144845
+
+        Reviewed by Martin Robinson.
+
+        * Source/cmake/OptionsGTK.cmake: Only define MOZ_X11 and XP_UNIX
+        if their corresponding WTF options are ON. The code in npapi.h
+        relies on these being undefined if they are to be switched off.
+
 2015-05-09  Yoav Weiss  <yoav@yoav.ws>
 
         Remove the PICTURE_SIZES build flag
index 4d0e9d8..c1654ef 100644 (file)
@@ -188,10 +188,17 @@ WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_THREADED_COMPOSITOR PRIVATE OFF)
 WEBKIT_OPTION_END()
 
 SET_AND_EXPOSE_TO_BUILD(WTF_PLATFORM_X11 ${ENABLE_X11_TARGET})
-SET_AND_EXPOSE_TO_BUILD(MOZ_X11 ${ENABLE_X11_TARGET})
-SET_AND_EXPOSE_TO_BUILD(XP_UNIX ${WTF_OS_UNIX})
 SET_AND_EXPOSE_TO_BUILD(WTF_PLATFORM_WAYLAND ${ENABLE_WAYLAND_TARGET})
 
+# MOZ_X11 and XP_UNIX are required by npapi.h. Their value is not checked;
+# only their definedness is. They should only be defined in the true case.
+if (${ENABLE_X11_TARGET})
+    SET_AND_EXPOSE_TO_BUILD(MOZ_X11 1)
+endif ()
+if (${WTF_OS_UNIX})
+    SET_AND_EXPOSE_TO_BUILD(XP_UNIX 1)
+endif ()
+
 set(USE_UDIS86 ON)
 set(ENABLE_WEBKIT OFF)
 set(ENABLE_WEBKIT2 ON)