summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorAmaury Pouly <amaury.pouly@gmail.com>2017-05-12 19:50:56 +1000
committerAmaury Pouly <amaury.pouly@gmail.com>2017-05-12 20:28:57 +1000
commitb959655822df6c5610372c9da54ea93f93ac87e9 (patch)
treef24104fb9da7a911ba7c9fb8a3e0137ddc7d1eac /tools
parent65c6a14e5fa41906176f93aa04512508e719d970 (diff)
downloadrockbox-b959655822df6c5610372c9da54ea93f93ac87e9.tar.gz
rockbox-b959655822df6c5610372c9da54ea93f93ac87e9.tar.bz2
rockbox-b959655822df6c5610372c9da54ea93f93ac87e9.zip
Fix unsafe substitutions in Makefile.
Strange things can happen in the (unlikely) case that ROOTDIR=/rockbox Change-Id: I085f928fd859b307667e8fccf40b29a9c325f7ae
Diffstat (limited to 'tools')
-rw-r--r--tools/functions.make15
-rw-r--r--tools/root.make2
2 files changed, 16 insertions, 1 deletions
diff --git a/tools/functions.make b/tools/functions.make
index 2a26e48bf2..806e1a2f3e 100644
--- a/tools/functions.make
+++ b/tools/functions.make
@@ -73,3 +73,18 @@ ifeq ($(call info),old)
export info=echo "$$(1)";
endif
+# path substituion can be surprisingly tricky, indeed the following almost
+# always work but not quite:
+# BMPOBJ = $(BMP:$(ROOTDIR)/%.bmp=$(BUILDDIR)/%.o)
+# indeed if
+# ROOTDIR=/rockbox
+# BUILDDIR=/rockbox/build-bla
+# then:
+# /rockbox/apps/bitmaps/rockboxlogo-bla.bmp -> /rockbox/apps/bitmaps/rockbox/build-blalogo.o
+# this function ensure that this type of problems does not occur
+#
+# usage: $(call full_path_subst,patterm,replacement,text)
+#
+# example: $(call full_path_subst,$(ROOTDIR)/%.bmp,$(BUILDDIR)/%.o,$(BMP))
+NO_MATCH=this_string_will_never_match_anything
+full_path_subst=$(patsubst $(NO_MATCH)/%,%, $(patsubst $(NO_MATCH)/$(1), $(2), $(addprefix $(NO_MATCH)/, $(3))))
diff --git a/tools/root.make b/tools/root.make
index 2deae8cc82..c9f4a2eb65 100644
--- a/tools/root.make
+++ b/tools/root.make
@@ -157,7 +157,7 @@ endif # CORE_GCSECTIONS
OBJ := $(SRC:.c=.o)
OBJ := $(OBJ:.S=.o)
OBJ += $(BMP:.bmp=.o)
-OBJ := $(subst $(ROOTDIR),$(BUILDDIR),$(OBJ))
+OBJ := $(call full_path_subst,$(ROOTDIR)/%,$(BUILDDIR)/%,$(OBJ))
build: $(TOOLS) $(BUILDDIR)/$(BINARY) $(CODECS) $(ROCKS) $(ARCHOSROM) $(RBINFO)