[CMake] Don't force-enable response files when using Ninja generator
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 8 Apr 2017 22:44:39 +0000 (22:44 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 8 Apr 2017 22:44:39 +0000 (22:44 +0000)
https://bugs.webkit.org/show_bug.cgi?id=170105

Patch by Ting-Wei Lan <lantw44@gmail.com> on 2017-04-08
Reviewed by Michael Catanzaro.

Not all platforms support response files, and unconditionally enabling
response files is known to cause build failure for some platforms.
Since WebKit builds fine on many platforms without force-enabling
response files and bug 129771 didn't mention which platform required
it, we remove it instead of adding more platform checks.

* Source/cmake/OptionsCommon.cmake:

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

ChangeLog
Source/cmake/OptionsCommon.cmake

index 394d4b6..345cb26 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2017-04-08  Ting-Wei Lan  <lantw44@gmail.com>
+
+        [CMake] Don't force-enable response files when using Ninja generator
+        https://bugs.webkit.org/show_bug.cgi?id=170105
+
+        Reviewed by Michael Catanzaro.
+
+        Not all platforms support response files, and unconditionally enabling
+        response files is known to cause build failure for some platforms.
+        Since WebKit builds fine on many platforms without force-enabling
+        response files and bug 129771 didn't mention which platform required
+        it, we remove it instead of adding more platform checks.
+
+        * Source/cmake/OptionsCommon.cmake:
+
 2017-04-07  Michael Catanzaro  <mcatanzaro@igalia.com>
 
         [GTK] Various build errors when plugin support is disabled
index a35b6d7..0396719 100644 (file)
@@ -208,18 +208,6 @@ if (NOT PORT STREQUAL "GTK")
     set(LIBEXEC_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/bin" CACHE PATH "Absolute path to install executables executed by the library")
 endif ()
 
-# The Ninja generator does not yet know how to build archives in pieces, and so response
-# files must be used to deal with very long linker command lines.
-# See https://bugs.webkit.org/show_bug.cgi?id=129771
-# The Apple Toolchain doesn't support response files.
-if (NOT APPLE)
-   # If using Ninja with cmake >= 3.6.0 and icecream, then the build is broken
-   # if enable the response files. See https://bugs.webkit.org/show_bug.cgi?id=168770
-   if (NOT ((((${CMAKE_CXX_COMPILER} MATCHES ".*ccache.*") AND ($ENV{CCACHE_PREFIX} MATCHES ".*icecc.*")) OR (${CMAKE_CXX_COMPILER} MATCHES ".*icecc.*")) AND (CMAKE_GENERATOR STREQUAL "Ninja") AND (${CMAKE_VERSION} VERSION_GREATER 3.5)))
-       set(CMAKE_NINJA_FORCE_RESPONSE_FILE 1)
-   endif ()
-endif ()
-
 # Macros for determining HAVE values.
 include(CheckIncludeFile)
 include(CheckFunctionExists)