[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen staging] xen: remove the usage of the P ar option



commit 1516ecd6f55fe3608f374f4f2548491472d1c9a1
Author:     Roger Pau Monne <roger.pau@xxxxxxxxxx>
AuthorDate: Wed Dec 30 18:34:46 2020 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Thu Dec 31 16:19:00 2020 +0000

    xen: remove the usage of the P ar option
    
    It's not part of the POSIX standard [0] and as such non GNU ar
    implementations don't usually have it.
    
    It's not relevant for the use case here anyway, as the archive file is
    recreated every time due to the rm invocation before the ar call. No
    file name matching should happen so matching using the full path name
    or a relative one should yield the same result.
    
    This fixes the build on FreeBSD.
    
    While there also drop the s option, as ar will already generate a
    symbol table by default when creating the archive.
    
    [0] https://pubs.opengroup.org/onlinepubs/9699919799/utilities/ar.html
    
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
 xen/Rules.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/Rules.mk b/xen/Rules.mk
index aba6ca2a90..ceb3d204b8 100644
--- a/xen/Rules.mk
+++ b/xen/Rules.mk
@@ -71,7 +71,7 @@ cmd_ld = $(LD) $(XEN_LDFLAGS) -r -o $@ $(filter-out 
%.a,$(real-prereqs)) \
 # ---------------------------------------------------------------------------
 
 quiet_cmd_ar = AR      $@
-cmd_ar = rm -f $@; $(AR) cPrs $@ $(real-prereqs)
+cmd_ar = rm -f $@; $(AR) cr $@ $(real-prereqs)
 
 # Objcopy
 # ---------------------------------------------------------------------------
--
generated by git-patchbot for /home/xen/git/xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.