[EFL] Clean up the use of DATA_DIR in the buildsystem
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Nov 2011 05:43:02 +0000 (05:43 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 18 Nov 2011 05:43:02 +0000 (05:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=72681

Patch by Raphael Kubo da Costa <kubo@profusion.mobi> on 2011-11-17
Reviewed by Daniel Bates.

.:

* Source/cmake/OptionsEfl.cmake: Rename DATA_DIR to DATA_INSTALL_DIR
and add it to the cache; add a variable with the path of the generated
theme and remove the -DDATA_DIR definition, it was moved to
PlatformEFL.cmake in WebCore.

Source/WebCore:

Add the -DDATA_DIR definition here instead of defining it globally in
OptionsEfl.cmake, as WebCore is the only place which needs it.

No new tests, this is a buildsystem change.

* PlatformEfl.cmake:

Source/WebKit/efl:

Instead of messing with BUILD_DATA_DIR and PARENT_SCOPE, just use
THEME_BINARY_DIR as defined in OptionsEfl.cmake.

* CMakeListsEfl.txt:

Tools:

Instead of adding -DDATA_DIR in all scopes, only define it in
WebCore/PlatformEFL.cmake, as WebCore is the only place that uses
it.

We can then define DATA_DIR to the generated theme directory for
EWebLauncher, since it's what we want here.

* CMakeListsEfl.txt:

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

ChangeLog
Source/WebCore/ChangeLog
Source/WebCore/PlatformEfl.cmake
Source/WebKit/efl/CMakeListsEfl.txt
Source/WebKit/efl/ChangeLog
Source/cmake/OptionsEfl.cmake
Tools/CMakeListsEfl.txt
Tools/ChangeLog

index d461352dee4a0b72ebfb068e0febdb7aecc820c2..3e0ea334d5baa984165b422515ecd870ec35f628 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2011-11-17  Raphael Kubo da Costa  <kubo@profusion.mobi>
+
+        [EFL] Clean up the use of DATA_DIR in the buildsystem
+        https://bugs.webkit.org/show_bug.cgi?id=72681
+
+        Reviewed by Daniel Bates.
+
+        * Source/cmake/OptionsEfl.cmake: Rename DATA_DIR to DATA_INSTALL_DIR
+        and add it to the cache; add a variable with the path of the generated
+        theme and remove the -DDATA_DIR definition, it was moved to
+        PlatformEFL.cmake in WebCore.
+
 2011-11-17  Patrick Gansterer  <paroga@webkit.org>
 
         [CMake] Add ENABLE_CLIENT_BASED_GEOLOCATION to cmakeconfig.h
index b6d26f5bb3757700a88d6db5122be34e757cbda8..af610d033adc5f5f5039a7ba1e23d3b696e079dc 100644 (file)
@@ -1,3 +1,17 @@
+2011-11-17  Raphael Kubo da Costa  <kubo@profusion.mobi>
+
+        [EFL] Clean up the use of DATA_DIR in the buildsystem
+        https://bugs.webkit.org/show_bug.cgi?id=72681
+
+        Reviewed by Daniel Bates.
+
+        Add the -DDATA_DIR definition here instead of defining it globally in
+        OptionsEfl.cmake, as WebCore is the only place which needs it.
+
+        No new tests, this is a buildsystem change.
+
+        * PlatformEfl.cmake:
+
 2011-11-17  Adam Klein  <adamk@chromium.org>
 
         Move JS recursion counter from V8Proxy to V8BindingPerIsolateData
index 665ad62e0b8a41a46d8ac2d0e16f86fd532c3b8e..5725930f202cede8331c92bd48120b1c8f1dbacd 100644 (file)
@@ -280,4 +280,5 @@ IF (WTF_USE_CURL)
   )
 ENDIF ()
 
-ADD_DEFINITIONS(-DWTF_USE_CROSS_PLATFORM_CONTEXT_MENUS=1)
+ADD_DEFINITIONS(-DWTF_USE_CROSS_PLATFORM_CONTEXT_MENUS=1
+                -DDATA_DIR="${CMAKE_INSTALL_PREFIX}/${DATA_INSTALL_DIR}")
index 20d532ac55161e7a477493945b4a89d7192ddecf..b0a63175f6c586ee83672adf255531e50a8caae6 100644 (file)
@@ -134,16 +134,13 @@ IF (WTF_USE_CURL)
   LIST(APPEND WebKit_LIBRARIES ${CURL_LIBRARIES} ${ZLIB_LIBRARIES})
 ENDIF ()
 
-SET(BUILD_DATA_DIR ${CMAKE_BINARY_DIR}/WebKit/efl/DefaultTheme)
-SET(BUILD_DATA_DIR ${BUILD_DATA_DIR} PARENT_SCOPE)
-FILE(MAKE_DIRECTORY ${BUILD_DATA_DIR})
-
 SET(WebKit_THEME_DEFINITION "")
 IF (ENABLE_PROGRESS_TAG)
   LIST(APPEND WebKit_THEME_DEFINITION "-DENABLE_PROGRESS_TAG")
 ENDIF ()
 
-SET(WebKit_THEME ${BUILD_DATA_DIR}/default.edj)
+FILE(MAKE_DIRECTORY ${THEME_BINARY_DIR})
+SET(WebKit_THEME ${THEME_BINARY_DIR}/default.edj)
 ADD_CUSTOM_COMMAND(
   OUTPUT ${WebKit_THEME}
   COMMAND ${EDJE_CC_EXECUTABLE} -v -id ${WEBKIT_DIR}/efl/DefaultTheme ${WebKit_THEME_DEFINITION} ${WEBKIT_DIR}/efl/DefaultTheme/default.edc ${WebKit_THEME}
@@ -260,4 +257,4 @@ INSTALL(FILES ${EWebKit_HEADERS}
         DESTINATION include/${WebKit_LIBRARY_NAME}-${PROJECT_VERSION_MAJOR})
 
 INSTALL(FILES ${WebKit_THEME}
-        DESTINATION share/${WebKit_LIBRARY_NAME}-${PROJECT_VERSION_MAJOR}/themes)
+        DESTINATION ${DATA_INSTALL_DIR}/themes)
index b49e33852d1e205b16c700a9e172c044032b2bed..24d92c4ffd687cdfaafdfda15ed79ab988806c69 100755 (executable)
@@ -1,3 +1,15 @@
+2011-11-17  Raphael Kubo da Costa  <kubo@profusion.mobi>
+
+        [EFL] Clean up the use of DATA_DIR in the buildsystem
+        https://bugs.webkit.org/show_bug.cgi?id=72681
+
+        Reviewed by Daniel Bates.
+
+        Instead of messing with BUILD_DATA_DIR and PARENT_SCOPE, just use
+        THEME_BINARY_DIR as defined in OptionsEfl.cmake.
+
+        * CMakeListsEfl.txt:
+
 2011-11-16  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
 
         [EFL] Use standard booleand data type in struct.
index 1756f350a434cdfa1676498e027aff751b61dabe..ed57993ccbaf4c3a68575f63e9263495f185fadf 100644 (file)
@@ -54,8 +54,8 @@ SET(JavaScriptCore_LIBRARY_NAME javascriptcore_efl)
 SET(WebCore_LIBRARY_NAME webcore_efl)
 SET(WebKit_LIBRARY_NAME ewebkit)
 
-SET(DATA_DIR ${CMAKE_INSTALL_PREFIX}/share/${WebKit_LIBRARY_NAME}-${PROJECT_VERSION_MAJOR})
-ADD_DEFINITIONS(-DDATA_DIR="${DATA_DIR}")
+SET(DATA_INSTALL_DIR "share/${WebKit_LIBRARY_NAME}-${PROJECT_VERSION_MAJOR}" CACHE PATH "Installation path for theme data")
+SET(THEME_BINARY_DIR ${CMAKE_BINARY_DIR}/WebKit/efl/DefaultTheme)
 
 SET(VERSION_SCRIPT "-Wl,--version-script,${CMAKE_MODULE_PATH}/eflsymbols.filter")
 
index 6b334ef45986fd8bfeedb9be4bee373a6ce43ab2..d1c43c36a20442a0d0b00893bb04f1636fc23b89 100644 (file)
@@ -49,9 +49,7 @@ IF (WTF_USE_CURL)
   LIST(APPEND EWebLauncher_LINK_FLAGS ${CURL_LDFLAGS})
 ENDIF ()
 
-# Override data directory. We always want to get a fresh theme.
-REMOVE_DEFINITIONS(-DDATA_DIR=\"${DATA_DIR}\")
-ADD_DEFINITIONS(-DDATA_DIR=\"${BUILD_DATA_DIR}\")
+ADD_DEFINITIONS(-DDATA_DIR=\"${THEME_BINARY_DIR}\")
 
 INCLUDE_DIRECTORIES(${EWebLauncher_INCLUDE_DIRECTORIES})
 ADD_EXECUTABLE(Programs/EWebLauncher ${EWebLauncher_SOURCES})
index 469c1cdbc31ced84edf2346bdd6920794bae25f9..bb619ba0597c5bda8c665e451b02091619dae078 100644 (file)
@@ -1,3 +1,19 @@
+2011-11-17  Raphael Kubo da Costa  <kubo@profusion.mobi>
+
+        [EFL] Clean up the use of DATA_DIR in the buildsystem
+        https://bugs.webkit.org/show_bug.cgi?id=72681
+
+        Reviewed by Daniel Bates.
+
+        Instead of adding -DDATA_DIR in all scopes, only define it in
+        WebCore/PlatformEFL.cmake, as WebCore is the only place that uses
+        it.
+
+        We can then define DATA_DIR to the generated theme directory for
+        EWebLauncher, since it's what we want here.
+
+        * CMakeListsEfl.txt:
+
 2011-11-17  Raphael Kubo da Costa  <kubo@profusion.mobi>
 
         [EFL] Move platform includes last in DRT's include directories.