freemyipod r4 - Code Review

Repository:	freemyipod
Revision:	< r3‎ \| r4 \| r5 >
Date:	02:13, 28 July 2010
Author:	farthen
Status:	new
Tags:
Comment:	Finally fixes the dependency creation bug in the Makefile. This needs to be tested under Linux.
Modified paths:	/embios/trunk/Makefile (modified) (history)

Diff [purge]

Index: embios/trunk/Makefile
—	—	@@ -41,7 +41,9 @@
42	42	@$(CC) -MM $(CFLAGS) -Itarget/$(1) -D TARGET=$(1) $$< > $$@.dep
43	43	@mv -f $$@.dep $$@.dep.tmp
44	44	@sed -e "s\|.*:\|$$@:\|" < $$@.dep.tmp > $$@.dep
45		~~- @sed -e "s/.://" -e "s/\\\\$$$$//" < $$@.dep.tmp \| fmt -1 \| sed -e "s/^ //" -e "s/$$$$/:/" >> $$@.dep~~
	45	+ @# Those many backslashes are necessary as make escapes them itself, then the shell
	46	+ @# So make makes 4 out of 8, then the shell makes 2 out of 4 which is what we want
	47	+ @sed -e "s/.://" -e "s/\\\\\\\\$$$$//" < $$@.dep.tmp \| fmt -1 \| sed -e "s/^ //" -e "s/$$$$/:/" >> $$@.dep
46	48	@rm -f $$@.dep.tmp
47	49
48	50	build/$(1)/%.o: %.S
—	—	@@ -54,7 +56,9 @@
55	57	@$(CC) -MM $(CFLAGS) -Itarget/$(1) -D TARGET=$(1) $$< > $$@.dep
56	58	@mv -f $$@.dep $$@.dep.tmp
57	59	@sed -e "s\|.*:\|$$@:\|" < $$@.dep.tmp > $$@.dep
58		~~- @sed -e "s/.://" -e "s/\\\\$$$$//" < $$@.dep.tmp \| fmt -1 \| sed -e "s/^ //" -e "s/$$$$/:/" >> $$@.dep~~
	60	+ @# Those many backslashes are necessary as make escapes them itself, then the shell
	61	+ @# So make makes 4 out of 8, then the shell makes 2 out of 4 which is what we want
	62	+ @sed -e "s/.://" -e "s/\\\\\\\\$$$$//" < $$@.dep.tmp \| fmt -1 \| sed -e "s/^ //" -e "s/$$$$/:/" >> $$@.dep
59	63	@rm -f $$@.dep.tmp
60	64	endef
61	65