Unreviewed, rolling out r168304.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 May 2014 13:34:58 +0000 (13:34 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 May 2014 13:34:58 +0000 (13:34 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132607

Broke the build (Requested by KaL on #webkit).

Reverted changeset:

"[GTK][CMake] Unable to do make install"
https://bugs.webkit.org/show_bug.cgi?id=130188
http://trac.webkit.org/changeset/168304

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

ChangeLog
Source/PlatformGTK.cmake
Source/cmake/OptionsGTK.cmake
Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index 1ee7009..37537fd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2014-05-06  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r168304.
+        https://bugs.webkit.org/show_bug.cgi?id=132607
+
+        Broke the build (Requested by KaL on #webkit).
+
+        Reverted changeset:
+
+        "[GTK][CMake] Unable to do make install"
+        https://bugs.webkit.org/show_bug.cgi?id=130188
+        http://trac.webkit.org/changeset/168304
+
 2014-05-05  Martin Robinson  <mrobinson@igalia.com>
 
         [GTK][CMake] Unable to do make install
index a957a8a..d937e47 100644 (file)
@@ -9,9 +9,6 @@ if (ENABLE_WEBKIT2)
         "${CMAKE_SOURCE_DIR}/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-docs.sgml"
         "${CMAKE_SOURCE_DIR}/Source/WebKit2/UIProcess/API/gtk/docs/webkit2gtk-sections.txt"
     )
-endif ()
-
-if (ENABLE_GTKDOC)
     install(DIRECTORY ${CMAKE_BINARY_DIR}/Documentation/webkit2gtk/html/
             DESTINATION ${CMAKE_INSTALL_DATADIR}/gtk-doc/html/webkit2gtk
     )
@@ -28,14 +25,14 @@ macro(ADD_GTKDOC_GENERATOR _stamp_name _extra_args)
 endmacro()
 
 add_gtkdoc_generator("docs-build-no-html.stamp" "--skip-html")
-add_custom_target(gtkdoc-no-html DEPENDS "${CMAKE_BINARY_DIR}/docs-build-no-html.stamp")
+add_custom_target(gtkdoc-no-html ALL
+    DEPENDS "${CMAKE_BINARY_DIR}/docs-build-no-html.stamp"
+)
 
 add_gtkdoc_generator("docs-build.stamp" "")
-if (ENABLE_GTKDOC)
-    add_custom_target(gtkdoc ALL DEPENDS "${CMAKE_BINARY_DIR}/docs-build.stamp")
-else ()
-    add_custom_target(gtkdoc DEPENDS "${CMAKE_BINARY_DIR}/docs-build.stamp")
-endif ()
+add_custom_target(gtkdoc
+    DEPENDS "${CMAKE_BINARY_DIR}/docs-build.stamp"
+)
 
 add_custom_target(check
     COMMAND ${TOOLS_DIR}/Scripts/run-gtk-tests
index 35fd4c7..f45cc2d 100644 (file)
@@ -17,7 +17,6 @@ CALCULATE_LIBRARY_VERSIONS_FROM_LIBTOOL_TRIPLE(WEBKIT2 32 0 7)
 CALCULATE_LIBRARY_VERSIONS_FROM_LIBTOOL_TRIPLE(JAVASCRIPTCORE 16 2 16)
 
 set(ENABLE_CREDENTIAL_STORAGE ON CACHE BOOL "Whether or not to enable support for credential storage using libsecret.")
-set(ENABLE_GTKDOC OFF CACHE BOOL "Whether or not to generate gtkdoc by default.")
 
 # FIXME: We want to expose fewer options to downstream, but for now everything is public.
 WEBKIT_OPTION_BEGIN()
index d58f7e7..e40f4fe 100644 (file)
@@ -1,3 +1,16 @@
+2014-05-06  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r168304.
+        https://bugs.webkit.org/show_bug.cgi?id=132607
+
+        Broke the build (Requested by KaL on #webkit).
+
+        Reverted changeset:
+
+        "[GTK][CMake] Unable to do make install"
+        https://bugs.webkit.org/show_bug.cgi?id=130188
+        http://trac.webkit.org/changeset/168304
+
 2014-05-04  Darin Adler  <darin@apple.com>
 
         RetainPtr: Use adoptCF function instead of AdoptCF constructor argument
index 5e32e21..17381f1 100755 (executable)
@@ -1875,9 +1875,7 @@ sub buildCMakeGeneratedProject($)
     if (isGtk()) {
         chdir "$buildPath" or die;
         $command = "$buildPath/build.sh";
-
-        # We always try to generate gtkdoc for GTK+, so that we see errors as early as possible.
-        @args = ($makeArgs, 'gtkdoc-no-html');
+        @args = ($makeArgs);
     }
 
     # We call system("cmake @args") instead of system("cmake", @args) so that @args is