Changeset 212882 in webkit
- Timestamp:
- Feb 23, 2017 3:27:10 AM (7 years ago)
- Location:
- trunk
- Files:
-
- 2 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/ChangeLog
r212875 r212882 1 2017-02-23 Tomas Popela <tpopela@redhat.com> 2 3 [GTK] Compilation fails if using ninja together with icecream and cmake > 3.5 4 https://bugs.webkit.org/show_bug.cgi?id=168770 5 6 Reviewed by Carlos Garcia Campos. 7 8 If using cmake >= 3.6 together with ninja generator and icecream, the 9 build will fail as icecream does not correctly handle the response 10 files and it's not passing compiler flags from there to the compiler 11 itself (in our case it's not passing -fPIC which leads to the 12 failure while linking). Don't enable the ninja's response files 13 support if we fulfill the preconditions. 14 15 * Source/cmake/OptionsCommon.cmake: 16 1 17 2017-02-22 Ryosuke Niwa <rniwa@webkit.org> 2 18 -
trunk/Source/cmake/OptionsCommon.cmake
r211635 r212882 214 214 # The Apple Toolchain doesn't support response files. 215 215 if (NOT APPLE) 216 set(CMAKE_NINJA_FORCE_RESPONSE_FILE 1) 216 # If using Ninja with cmake >= 3.6.0 and icecream, then the build is broken 217 # if enable the response files. See https://bugs.webkit.org/show_bug.cgi?id=168770 218 if (NOT ((${CMAKE_CXX_COMPILER} MATCHES ".*icecc.*") AND (CMAKE_GENERATOR STREQUAL "Ninja") AND (${CMAKE_VERSION} VERSION_GREATER 3.5))) 219 set(CMAKE_NINJA_FORCE_RESPONSE_FILE 1) 220 endif () 217 221 endif () 218 222
Note: See TracChangeset
for help on using the changeset viewer.