Patrick Georgi (pgeorgi@google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12558
-gerrit
commit e5fbe77d7633ff2519718eb687c30fb643230705 Author: Patrick Georgi pgeorgi@chromium.org Date: Fri Nov 27 12:36:28 2015 +0100
build system: actually provide revision information in defconfig
The config file added to CBFS is the short version created by defconfig. The build system tried to add a header describing the version for quite a while now, but failed because it wrote to the file, then had kconfig overwrite it with the config data.
While at it, rely on build.h and its version information instead of calling git manually.
Change-Id: I5e4d6c857594a55432c05bf1480973fc950f4d4a Signed-off-by: Patrick Georgi pgeorgi@chromium.org --- Makefile.inc | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/Makefile.inc b/Makefile.inc index d5cf337..04aa076 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -228,10 +228,13 @@ cbfs-files-processor-vsa= \ # arg1: input # arg2: output cbfs-files-processor-defconfig= \ - $(eval $(2): $(1) ; \ + $(eval $(2): $(1) $(obj)/build.h; \ printf " CREATE $(2) (from $(1))\n"; \ - echo "# This image was built using git revision" `git rev-parse HEAD` > $(2).tmp && \ - $(MAKE) DOTCONFIG=$(1) DEFCONFIG=$(2).tmp savedefconfig && \ + printf "# This image was built using coreboot " > $(2).tmp && \ + grep "<COREBOOT_VERSION>" $(obj)/build.h |cut -d" -f2 >> $(2).tmp && \ + $(MAKE) DOTCONFIG=$(1) DEFCONFIG=$(2).tmp2 savedefconfig && \ + cat $(2).tmp2 >> $(2).tmp && \ + rm -f $(2).tmp2 && \ \mv -f $(2).tmp $(2))
#######################################################################