[coreboot-gerrit] Patch set updated for coreboot: Makefile: Don't copy thin archives around
Nico Huber (nico.h@gmx.de)
gerrit at coreboot.org
Mon Jan 25 01:39:07 CET 2016
Nico Huber (nico.h at gmx.de) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13179
-gerrit
commit 3ae87016849d6f5beef841157272805f30aea0bc
Author: Nico Huber <nico.h at gmx.de>
Date: Sat Jan 23 03:23:23 2016 +0100
Makefile: Don't copy thin archives around
We can't just copy archives around as they may be thin archives which
contain relative paths. Using ar to create another thin archive should
result in the same archive with fixed paths.
Tested by verifying that the resulting coreboot.rom files didn't change
for all of Jenkins' abuild configurations.
Change-Id: Ic5743da2f4b5eb246fafd02181d66c5d40e7f00c
Signed-off-by: Nico Huber <nico.h at gmx.de>
---
Makefile.inc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile.inc b/Makefile.inc
index 89f02d4..8bf0ca5 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -179,8 +179,8 @@ endef
define generic-objs_a_template_gen
de$(EMPTY)fine $(1)-objs_a_template
$$(call src-to-obj,$1,$$(1).a): $$(1).a
- @printf " CP $$$$(subst $$$$(obj)/,,$$$$(@))\n"
- cp $$$$< $$$$@.tmp
+ @printf " AR $$$$(subst $$$$(obj)/,,$$$$(@))\n"
+ $$$$(AR_$(1)) rcsT $$$$@.tmp $$$$<
mv $$$$@.tmp $$$$@
en$(EMPTY)def
endef
More information about the coreboot-gerrit
mailing list