2010-07-07 Rafael Antognolli <antognolli@profusion.mobi>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jul 2010 14:23:50 +0000 (14:23 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jul 2010 14:23:50 +0000 (14:23 +0000)
        Reviewed by Adam Barth.

        [EFL] Fix cmake build and libsoup detection
        Some changes should be done to cmake build system to fix the detection of libsoup and correct some link paths:
        - LINK_FLAGS should be set using quotes, otherwise some wrong
        substitution takes place;
        - we should use LIBSOUP24_* instead of LIBSOUP_* since this is the
        version we are using;
        - need to set HAVE_LIBSOUP_2_29_90 if we find a version equal or newer
        than that.
        https://bugs.webkit.org/show_bug.cgi?id=41717

        * cmake/OptionsEfl.cmake:
2010-07-07  Rafael Antognolli  <antognolli@profusion.mobi>

        Reviewed by Adam Barth.

        [EFL] Fix cmake build and libsoup detection
        Some changes should be done to cmake build system to fix the detection of libsoup and correct some link paths:
        - LINK_FLAGS should be set using quotes, otherwise some wrong
        substitution takes place;
        - we should use LIBSOUP24_* instead of LIBSOUP_* since this is the
        version we are using;
        - need to set HAVE_LIBSOUP_2_29_90 if we find a version equal or newer
        than that.
        https://bugs.webkit.org/show_bug.cgi?id=41717

        * CMakeLists.txt:
        * CMakeListsEfl.txt:

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

ChangeLog
WebKit/CMakeLists.txt
WebKit/CMakeListsEfl.txt
WebKit/ChangeLog
cmake/OptionsEfl.cmake

index 7a2fc5f9dac081d036605f0d1a7cdf8acd8dff7a..bd29c18eba46ddcf6b2b1600bb80a301eed509e9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2010-07-07  Rafael Antognolli  <antognolli@profusion.mobi>
+
+        Reviewed by Adam Barth.
+
+        [EFL] Fix cmake build and libsoup detection
+        Some changes should be done to cmake build system to fix the detection of libsoup and correct some link paths:
+        - LINK_FLAGS should be set using quotes, otherwise some wrong
+        substitution takes place;
+        - we should use LIBSOUP24_* instead of LIBSOUP_* since this is the
+        version we are using;
+        - need to set HAVE_LIBSOUP_2_29_90 if we find a version equal or newer
+        than that.
+        https://bugs.webkit.org/show_bug.cgi?id=41717
+
+        * cmake/OptionsEfl.cmake:
+
 2010-07-06  Leandro Pereira  <leandro@profusion.mobi>
 
         Unreviewed build fix.
index 5b0cebd2d8c5025e851a76c28e185676bdda03ed..256b276babb818dc7506d776edf824e40d828eec 100644 (file)
@@ -57,7 +57,7 @@ ADD_DEFINITIONS(-DBUILDING_WebKit)
 INCLUDE_DIRECTORIES(${WebKit_INCLUDE_DIRECTORIES})
 ADD_LIBRARY(${WebKit_LIBRARY_NAME} ${WebKit_LIBRARY_TYPE} ${WebKit_SOURCES})
 TARGET_LINK_LIBRARIES(${WebKit_LIBRARY_NAME} ${WebKit_LIBRARIES})
-ADD_TARGET_PROPERTIES(${WebKit_LIBRARY_NAME} LINK_FLAGS ${WebKit_LINK_FLAGS})
+ADD_TARGET_PROPERTIES(${WebKit_LIBRARY_NAME} LINK_FLAGS "${WebKit_LINK_FLAGS}")
 
 INSTALL(TARGETS ${WebKit_LIBRARY_NAME} DESTINATION lib)
 
index 7eeddebd0b18330ed0b720e6a054559d3a43f735..86d03278b8eacf616d3e0a2f820ce8c47927a039 100644 (file)
@@ -17,7 +17,7 @@ LIST(APPEND WebKit_INCLUDE_DIRECTORIES
     ${EVAS_INCLUDE_DIRS}
     ${Gdk_INCLUDE_DIRS}
     ${Glib_INCLUDE_DIRS}
-    ${LIBSOUP2_INCLUDE_DIRS}
+    ${LIBSOUP24_INCLUDE_DIRS}
     ${LIBXML2_INCLUDE_DIRS}
     ${LIBXSLT_INCLUDE_DIRS}
     ${SQLITE_INCLUDE_DIRS}
@@ -50,7 +50,7 @@ LIST(APPEND WebKit_LIBRARIES
     ${Gdk_LIBRARIES}
     ${Glib_LIBRARIES}
     ${Gtk_LIBRARIES}
-    ${LIBSOUP_LIBRARIES}
+    ${LIBSOUP24_LIBRARIES}
     ${LIBXML2_LIBRARIES}
     ${Pango_LIBRARIES}
     ${SQLITE_LIBRARIES}
@@ -79,7 +79,7 @@ SET(EWebLauncher_LIBRARIES
     ${EVAS_LIBRARIES}
     ${Gdk_LIBRARIES}
     ${Glib_LIBRARIES}
-    ${LIBSOUP2_LIBRARIES}
+    ${LIBSOUP24_LIBRARIES}
     ${LIBXML2_LIBRARIES}
     ${LIBXSLT_LIBRARIES}
     ${SQLITE_LIBRARIES}
@@ -94,7 +94,7 @@ SET(EWebLauncher_LINK_FLAGS
 
 ADD_EXECUTABLE(../Programs/EWebLauncher ${EWebLauncher_SOURCES})
 TARGET_LINK_LIBRARIES(../Programs/EWebLauncher ${EWebLauncher_LIBRARIES})
-ADD_TARGET_PROPERTIES(../Programs/EWebLauncher LINK_FLAGS ${EWebLauncher_LINK_FLAGS})
+ADD_TARGET_PROPERTIES(../Programs/EWebLauncher LINK_FLAGS "${EWebLauncher_LINK_FLAGS}")
 
 IF (SHARED_CORE)
     SET(LIBS_PRIVATE "-l${WTF_LIBRARY_NAME} -l${JavaScriptCore_LIBRARY_NAME} -l${WebCore_LIBRARY_NAME}")
index a987c9a95b13dff7c68532d351353d33843e70ef..b3f4d1d65fb0e8147ebaf7af3f56ce5e0bfca68b 100644 (file)
@@ -1,3 +1,20 @@
+2010-07-07  Rafael Antognolli  <antognolli@profusion.mobi>
+
+        Reviewed by Adam Barth.
+
+        [EFL] Fix cmake build and libsoup detection
+        Some changes should be done to cmake build system to fix the detection of libsoup and correct some link paths:
+        - LINK_FLAGS should be set using quotes, otherwise some wrong
+        substitution takes place;
+        - we should use LIBSOUP24_* instead of LIBSOUP_* since this is the
+        version we are using;
+        - need to set HAVE_LIBSOUP_2_29_90 if we find a version equal or newer
+        than that.
+        https://bugs.webkit.org/show_bug.cgi?id=41717
+
+        * CMakeLists.txt:
+        * CMakeListsEfl.txt:
+
 2010-07-05  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
 
         Reviewed by Kenneth Rohde Christiansen.
index ff65147bbbcde6c15d9391d6867b5e424b99aaf8..3408ea7c9430bd63b8cf9aa45e227c4570b12505 100644 (file)
@@ -34,9 +34,10 @@ ADD_DEFINITIONS(-DWTF_PLATFORM_CAIRO=1)
 
 SET(WTF_USE_SOUP 1)
 ADD_DEFINITIONS(-DWTF_USE_SOUP=1)
-IF (LIBSOUP_VERSION VERSION_LESS 2.29.90)
-ELSE (LIBSOUP_VERSION VERSION_LESS 2.29.90)
+IF (LIBSOUP24_VERSION VERSION_LESS 2.29.90)
+ELSE (LIBSOUP24_VERSION VERSION_LESS 2.29.90)
     SET(HAVE_LIBSOUP_2_29_90 1)
+    ADD_DEFINITIONS(-DHAVE_LIBSOUP_2_29_90=1)
 ENDIF ()
 
 SET(WTF_USE_FREETYPE 1)