diff mbox

btrfs-progs: makefile: drop u option from ar invocation

Message ID 1441407978-23061-1-git-send-email-arnd@arndnet.de (mailing list archive)
State Accepted
Headers show

Commit Message

Arnd Hannemann Sept. 4, 2015, 11:06 p.m. UTC
In newer distros (ubuntu 15.10, fedora rawhide) the binutils
ar uses the new D flag per default to build deterministic
binaries.
Without this patch the following warning is issued, when
building btrfs-progs:

    [AR]     libbtrfs.a
/usr/bin/ar: `u' modifier ignored since `D' is the default (see `U')

For libtrfs.a performance benefit of the u option can be neglected,
so drop the u option and silence the warning.

In the future one might want to explicitly add the D option anyway.

Signed-off-by: Arnd Hannemann <arnd@arndnet.de>
---
 Makefile.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Sterba Sept. 8, 2015, 11:56 a.m. UTC | #1
On Sat, Sep 05, 2015 at 01:06:18AM +0200, Arnd Hannemann wrote:
> In newer distros (ubuntu 15.10, fedora rawhide) the binutils
> ar uses the new D flag per default to build deterministic
> binaries.
> Without this patch the following warning is issued, when
> building btrfs-progs:
> 
>     [AR]     libbtrfs.a
> /usr/bin/ar: `u' modifier ignored since `D' is the default (see `U')
> 
> For libtrfs.a performance benefit of the u option can be neglected,
> so drop the u option and silence the warning.
> 
> In the future one might want to explicitly add the D option anyway.
> 
> Signed-off-by: Arnd Hannemann <arnd@arndnet.de>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Makefile.in b/Makefile.in
index 665f83c..514a76f 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -199,7 +199,7 @@  $(libs_shared): $(libbtrfs_objects) $(lib_links) send.h
 
 $(libs_static): $(libbtrfs_objects)
 	@echo "    [AR]     $@"
-	$(Q)$(AR) cru libbtrfs.a $(libbtrfs_objects)
+	$(Q)$(AR) cr libbtrfs.a $(libbtrfs_objects)
 
 $(lib_links):
 	@echo "    [LN]     $@"