'ar T' is not portable and breaks the build on FreeBSD

https://bugs.webkit.org/show_bug.cgi?id=128596

Patch by Ryan Lortie <desrt@desrt.ca> on 2014-02-12
Reviewed by Gustavo Noronha Silva.

Create thin archives only if we are using GNU ar.

* Source/autotools/SetupLibtool.m4:

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

diff --git a/ChangeLog b/ChangeLog
index 0ba3614..342ee3e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2014-02-12  Ryan Lortie  <desrt@desrt.ca>
+
+        'ar T' is not portable and breaks the build on FreeBSD
+        https://bugs.webkit.org/show_bug.cgi?id=128596
+
+        Reviewed by Gustavo Noronha Silva.
+
+        Create thin archives only if we are using GNU ar.
+
+        * Source/autotools/SetupLibtool.m4:
+
 2014-02-10  Krzysztof Wolanski  <k.wolanski@samsung.com>
 
         Add the CSS_SHAPE_INSIDE to CMake feature list

diff --git a/Source/autotools/SetupLibtool.m4 b/Source/autotools/SetupLibtool.m4
index 1c8feb0..05545ca 100644 (file)
--- a/Source/autotools/SetupLibtool.m4
+++ b/Source/autotools/SetupLibtool.m4
@@ -6,7 +6,11 @@ AC_SUBST([LIBJAVASCRIPTCOREGTK_VERSION])
 AC_SUBST([LIBWEBKIT2GTK_VERSION])
 
 if test -z "$AR_FLAGS"; then
-    AR_FLAGS="cruT"
+    if "${AR:-ar}" -V | grep -q 'GNU ar'; then
+        AR_FLAGS="cruT"
+    else
+        AR_FLAGS="cru"
+    fi
 fi
 AC_SUBST([AR_FLAGS])