[EFL] Use SET_AND_EXPOSE_TO_BUILD instead of definitions
authorryuan.choi@navercorp.com <ryuan.choi@navercorp.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Jun 2015 23:35:24 +0000 (23:35 +0000)
committerryuan.choi@navercorp.com <ryuan.choi@navercorp.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Jun 2015 23:35:24 +0000 (23:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=146108

Reviewed by Gyuyoung Kim.

* Source/cmake/OptionsEfl.cmake:

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

ChangeLog
Source/cmake/OptionsEfl.cmake

index 5de22f6..bfd2d14 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2015-06-18  Ryuan Choi  <ryuan.choi@navercorp.com>
+
+        [EFL] Use SET_AND_EXPOSE_TO_BUILD instead of definitions
+        https://bugs.webkit.org/show_bug.cgi?id=146108
+
+        Reviewed by Gyuyoung Kim.
+
+        * Source/cmake/OptionsEfl.cmake:
+
 2015-06-16  Ryuan Choi  <ryuan.choi@navercorp.com>
 
         [EFL] Remove duplicated PUBLIC argument in OptionsEfl.cmake
index ada4988..1a1896a 100644 (file)
@@ -142,9 +142,9 @@ if (ENABLE_ECORE_X)
     find_package(X11 REQUIRED)
 
     list(APPEND ECORE_ADDITIONAL_COMPONENTS X)
-    add_definitions(-DHAVE_ECORE_X)
-    add_definitions(-DWTF_PLATFORM_X11=1)
-    add_definitions(-DMOZ_X11)
+    SET_AND_EXPOSE_TO_BUILD(HAVE_ECORE_X 1)
+    SET_AND_EXPOSE_TO_BUILD(WTF_PLATFORM_X11 1)
+    SET_AND_EXPOSE_TO_BUILD(MOZ_X11 1)
 endif ()
 
 if (ENABLE_ACCESSIBILITY)
@@ -186,7 +186,7 @@ endif ()
 
 if (ENABLE_MEDIA_STREAM)
     find_package(OpenWebRTC REQUIRED)
-    add_definitions(-DUSE_OPENWEBRTC)
+    SET_AND_EXPOSE_TO_BUILD(USE_OPENWEBRTC TRUE)
 endif ()
 
 if (ENABLE_NETSCAPE_PLUGIN_API)
@@ -195,13 +195,12 @@ endif ()
 
 if (WTF_OS_UNIX)
     set(ENABLE_X11_TARGET ON)
-    add_definitions(-DXP_UNIX)
+    SET_AND_EXPOSE_TO_BUILD(XP_UNIX 1)
 endif (WTF_OS_UNIX)
 
 if (ENABLE_VIDEO OR ENABLE_WEB_AUDIO)
     set(GSTREAMER_COMPONENTS app mpegts pbutils)
-    set(USE_GSTREAMER 1)
-    add_definitions(-DUSE_GSTREAMER=1)
+    SET_AND_EXPOSE_TO_BUILD(USE_GSTREAMER TRUE)
 
     if (ENABLE_VIDEO)
         list(APPEND GSTREAMER_COMPONENTS video tag)
@@ -220,33 +219,28 @@ if (ENABLE_VIDEO OR ENABLE_WEB_AUDIO)
     endif ()
 endif ()
 
-add_definitions(-DUSE_COORDINATED_GRAPHICS=1)
-add_definitions(-DUSE_COORDINATED_GRAPHICS_MULTIPROCESS=1)
-add_definitions(-DUSE_TEXTURE_MAPPER=1)
-
-set(ENABLE_GRAPHICS_CONTEXT_3D 1)
-add_definitions(-DENABLE_GRAPHICS_CONTEXT_3D=1)
-add_definitions(-DUSE_GRAPHICS_SURFACE=1)
+SET_AND_EXPOSE_TO_BUILD(ENABLE_GRAPHICS_CONTEXT_3D TRUE)
+SET_AND_EXPOSE_TO_BUILD(USE_COORDINATED_GRAPHICS TRUE)
+SET_AND_EXPOSE_TO_BUILD(USE_COORDINATED_GRAPHICS_MULTIPROCESS TRUE)
+SET_AND_EXPOSE_TO_BUILD(USE_GRAPHICS_SURFACE TRUE)
+SET_AND_EXPOSE_TO_BUILD(USE_TEXTURE_MAPPER TRUE)
 
 option(ENABLE_GLES2 "Enable GLES Support")
 if (ENABLE_GLES2)
     find_package(OpenGLES2 REQUIRED)
-    set(USE_OPENGL_ES_2 1)
-    add_definitions(-DUSE_OPENGL_ES_2=1)
+    SET_AND_EXPOSE_TO_BUILD(USE_OPENGL_ES_2 TRUE)
 else ()
     find_package(OpenGL REQUIRED)
-    set(USE_OPENGL 1)
-    add_definitions(-DUSE_OPENGL=1)
+    SET_AND_EXPOSE_TO_BUILD(USE_OPENGL TRUE)
 endif ()
 
 option(ENABLE_EGL "Enable EGL Support")
 if (ENABLE_EGL)
     find_package(EGL REQUIRED)
-    set(USE_EGL 1)
-    add_definitions(-DUSE_EGL=1)
+    SET_AND_EXPOSE_TO_BUILD(USE_EGL TRUE)
 else ()
     CHECK_INCLUDE_FILES("GL/glx.h" OPENGLX_FOUND)
-    add_definitions(-DUSE_GLX=1)
+    SET_AND_EXPOSE_TO_BUILD(USE_GLX TRUE)
 endif ()
 
 if (ENABLE_WEBGL AND OPENGLX_FOUND)
@@ -266,7 +260,6 @@ endif ()
 
 if (ENABLE_SPEECH_SYNTHESIS)
     find_package(Espeak REQUIRED)
-    add_definitions(-DENABLE_SPEECH_SYNTHESIS=1)
 endif ()
 
 if (ENABLE_FTL_JIT)