[cmake] Don't add -fPIC flag when on Windows (MinGW)
authorannulen@yandex.ru <annulen@yandex.ru@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Oct 2016 17:27:00 +0000 (17:27 +0000)
committerannulen@yandex.ru <annulen@yandex.ru@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Oct 2016 17:27:00 +0000 (17:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163949

Reviewed by Michael Catanzaro.

-fPIC flag does not do anything on Windows and produces warning noise
with MinGW. There is no position independent code in Unix sense on
32-bit Windows, and 64-bit Windows code is always relocatable.

For reference see
https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00836.html

* Source/cmake/WebKitHelpers.cmake:

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

ChangeLog
Source/cmake/WebKitHelpers.cmake

index 7a06269..eab8854 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2016-10-25  Konstantin Tokarev  <annulen@yandex.ru>
+
+        [cmake] Don't add -fPIC flag when on Windows (MinGW)
+        https://bugs.webkit.org/show_bug.cgi?id=163949
+
+        Reviewed by Michael Catanzaro.
+
+        -fPIC flag does not do anything on Windows and produces warning noise
+        with MinGW. There is no position independent code in Unix sense on
+        32-bit Windows, and 64-bit Windows code is always relocatable.
+
+        For reference see
+        https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00836.html
+
+        * Source/cmake/WebKitHelpers.cmake:
+
 2016-10-21  Csaba Osztrogon√°c  <ossy@webkit.org>
 
         [EFL] REGRESSION(r207616): Build is broken due to missing libgeoclue-dev package
index 38ffbd5..2835fca 100644 (file)
@@ -10,9 +10,11 @@ macro(WEBKIT_SET_EXTRA_COMPILER_FLAGS _target)
             set(OLD_COMPILE_FLAGS "")
         endif ()
 
-        get_target_property(TARGET_TYPE ${_target} TYPE)
-        if (${TARGET_TYPE} STREQUAL "STATIC_LIBRARY") # -fPIC is automatically added to shared libraries
-            set(OLD_COMPILE_FLAGS "-fPIC ${OLD_COMPILE_FLAGS}")
+        if (NOT WIN32)
+            get_target_property(TARGET_TYPE ${_target} TYPE)
+            if (${TARGET_TYPE} STREQUAL "STATIC_LIBRARY") # -fPIC is automatically added to shared libraries
+                set(OLD_COMPILE_FLAGS "-fPIC ${OLD_COMPILE_FLAGS}")
+            endif ()
         endif ()
 
         # Suppress -Wparentheses-equality warning of Clang