[coreboot-gerrit] Change in coreboot[master]: Makefile: Don't rebuild when generating tags or file list

[Martin Roth (Code Review)]

Martin Roth has uploaded this change for review. ( https://review.coreboot.org/21585


Change subject: Makefile: Don't rebuild when generating tags or file list
......................................................................

Makefile: Don't rebuild when generating tags or file list

Generating a project file list used to do a rebuild of the project.
Instead, just make sure there's a coreboot.rom file present and if
it is, generate the list.

Change-Id: I9cc12ef3d1990c3422625630451b2a7b8d77829c
Signed-off-by: Martin Roth <martinroth at google.com>
---
M Makefile
1 file changed, 6 insertions(+), 3 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/85/21585/1

diff --git a/Makefile b/Makefile
index 383d583..4ad0177 100644
--- a/Makefile
+++ b/Makefile
@@ -369,13 +369,16 @@
 $(shell mkdir -p $(KCONFIG_SPLITCONFIG) $(objk)/lxdialog $(additional-dirs) $(alldirs))
 endif
 
-$(obj)/project_filelist.txt: all
+$(obj)/project_filelist.txt:
+	if [ -z "$(wildcard $(obj)/coreboot.rom)" ]; then \
+		echo "*** Error: Project must be built before generating file list ***"; \
+		exit 1; \
+	fi
 	find $(obj) -name "*.d" -exec cat {} \; | \
 	  sed 's/[:\\]/ /g' | sed 's/ /\n/g' | sort | uniq | \
 	  grep -v '\.o$$' > $(obj)/project_filelist.txt
 
-filelist: clean
-	$(MAKE) $(obj)/project_filelist.txt
+filelist: $(obj)/project_filelist.txt
 	printf "\nFiles used in build:\n"
 	cat $(obj)/project_filelist.txt
 

-- 
To view, visit https://review.coreboot.org/21585
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9cc12ef3d1990c3422625630451b2a7b8d77829c
Gerrit-Change-Number: 21585
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170918/271cf144/attachment.html>