From: Pavel Raiskup Date: Wed, 1 Jul 2015 07:38:35 +0000 (+0200) Subject: gnulib-common.m4: change the ARFLAGS default to 'cr' X-Git-Tag: v1.0~7019 X-Git-Url: https://gitweb.git.savannah.gnu.org/gitweb/?a=commitdiff_plain;h=551098026bd1e4e10aeccc8dde40539fbc325082;p=gnulib.git gnulib-common.m4: change the ARFLAGS default to 'cr' In some GNU/Linux distributions people started to compile 'ar' binary with --enable-deterministic-archives (binutils project). That, however, in combination with previous autotools long time working default AR{_,}FLAGS=cru causes warnings on such installations: ar: `u' modifier ignored since `D' is the default (see `U') The 'u' option (at least with GNU binutils) did small optimization during repeated builds because it instructed 'ar' to not open/close unchanged *.o files and to rather read their contents from old archive file. However, its removal should not cause a big performance hit for usual workflows. Distributions started using --enable-deterministic-archives knowing that it will disable the 'u', with the benefit of having rather a bit more deterministic builds. Also, to justify this change a bit more, keeping 'u' in ARFLAGS could only result in many per-project changes to override Automake's ARFLAGS default, just to silent such warnings. * m4/gnulib-common.m4 (gl_PROG_AR_RANLIB): Set ARFLAGS='cr' if not set already. --- diff --git a/ChangeLog b/ChangeLog index 94852988ab..5cab00b876 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +2015-07-01 Pavel Raiskup + + gnulib-common.m4: change the ARFLAGS default to 'cr' + In some GNU/Linux distributions people started to compile 'ar' + binary with --enable-deterministic-archives (binutils project). + That, however, in combination with previous autotools long time + working default AR{_,}FLAGS=cru causes warnings on such + installations: + ar: `u' modifier ignored since `D' is the default (see `U') + * m4/gnulib-common.m4 (gl_PROG_AR_RANLIB): Set ARFLAGS='cr' if not + set already. + 2015-07-01 Pavel Raiskup selinux-h: avoid double free after *getfilecon() diff --git a/m4/gnulib-common.m4 b/m4/gnulib-common.m4 index b301abeda0..40e82f6e27 100644 --- a/m4/gnulib-common.m4 +++ b/m4/gnulib-common.m4 @@ -278,12 +278,12 @@ Amsterdam dnl but prefer ${host}-ar over ar (useful for cross-compiling). AC_CHECK_TOOL([AR], [ar], [ar]) if test -z "$ARFLAGS"; then - ARFLAGS='cru' + ARFLAGS='cr' fi fi else if test -z "$ARFLAGS"; then - ARFLAGS='cru' + ARFLAGS='cr' fi fi AC_SUBST([AR])