diff options
author | Solomon Peachy <pizza@shaftnet.org> | 2018-12-25 10:33:35 -0500 |
---|---|---|
committer | Solomon Peachy <pizza@shaftnet.org> | 2018-12-25 11:54:11 -0500 |
commit | 4c933a1d018cdd4d9528441652794971a8a806bc (patch) | |
tree | 96c779158dd73e95876b5306df3c72c2eed66c9c | |
parent | 9b3f22ac3af7f89f8b70aa2580435fbb9a5cf052 (diff) | |
download | rockbox-4c933a1d018cdd4d9528441652794971a8a806bc.tar.gz rockbox-4c933a1d018cdd4d9528441652794971a8a806bc.zip |
build: Properly generate build dependency for autogenerated bitmap .h files
* bmp2rb generated a .h file that rockbox .c files used.
* .h files in .c files were used to generate dependency graphs for make
* When Make saw the .h file for the bitmap, it didn't know how what
to do with them
* Only arose in parallel builds
Fixed this by adding explicit dependencies for the .h files as part of
the existing 'bmpdepfile' function.
Solves the Xduoo X3 bootloader build failure that I could trigger 100%
of the time by using 'make -j8'
Change-Id: I6b3e78dde26c820a3b6c7c286e7d6c981b8e01fc
-rw-r--r-- | tools/functions.make | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/functions.make b/tools/functions.make index 806e1a2f3e..a2c715e9e1 100644 --- a/tools/functions.make +++ b/tools/functions.make | |||
@@ -55,8 +55,10 @@ bmpdepfile = $(SILENT) \ | |||
55 | for each in $(2); do \ | 55 | for each in $(2); do \ |
56 | obj=`echo $$each | sed -e 's/\.bmp/.o/' -e 's:$(ROOTDIR):$(BUILDDIR):'`; \ | 56 | obj=`echo $$each | sed -e 's/\.bmp/.o/' -e 's:$(ROOTDIR):$(BUILDDIR):'`; \ |
57 | src=`echo $$each | sed -e 's/\.bmp/.c/' -e 's:$(ROOTDIR):$(BUILDDIR):'`; \ | 57 | src=`echo $$each | sed -e 's/\.bmp/.c/' -e 's:$(ROOTDIR):$(BUILDDIR):'`; \ |
58 | hdr=`echo $$each | sed -e 's/.*\/\(.*\)\..*\.bmp/bitmaps\/\1\.h/'`; \ | ||
58 | echo $$obj: $$src; \ | 59 | echo $$obj: $$src; \ |
59 | echo $$src: $$each; \ | 60 | echo $$src: $$each; \ |
61 | echo $(BUILDDIR)/$$hdr: $$src; \ | ||
60 | done \ | 62 | done \ |
61 | >> $(1) | 63 | >> $(1) |
62 | 64 | ||