From 0dbd928ceefed09d65276211c70d0137b4734011 Mon Sep 17 00:00:00 2001
From: Paul Sokolovsky <pfalcon@users.sourceforge.net>
Date: Sat, 24 Oct 2015 02:26:10 +0300
Subject: [PATCH] Makefiles: Remove duplicate object files when linking.

Scenario: module1 depends on some common file from lib/, so specifies it
in its SRC_MOD, and the same situation with module2, then common file
from lib/ eventually ends up listed twice in $(OBJ), which leads to link
errors.

Make is equipped to deal with such situation easily, quoting the manual:
"The value of $^ omits duplicate prerequisites, while $+ retains them and
preserves their order." So, just use $^ consistently in all link targets.
---
 bare-arm/Makefile | 2 +-
 esp8266/Makefile  | 2 +-
 minimal/Makefile  | 2 +-
 pic16bit/Makefile | 2 +-
 py/mkrules.mk     | 4 ++--
 qemu-arm/Makefile | 4 ++--
 stmhal/Makefile   | 2 +-
 teensy/Makefile   | 2 +-
 8 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/bare-arm/Makefile b/bare-arm/Makefile
index dc17eed4d..a57ccfd19 100644
--- a/bare-arm/Makefile
+++ b/bare-arm/Makefile
@@ -42,7 +42,7 @@ all: $(BUILD)/firmware.elf
 
 $(BUILD)/firmware.elf: $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(LD) $(LDFLAGS) -o $@ $(OBJ) $(LIBS)
+	$(Q)$(LD) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 include ../py/mkrules.mk
diff --git a/esp8266/Makefile b/esp8266/Makefile
index b06f6787a..7db5a76d3 100644
--- a/esp8266/Makefile
+++ b/esp8266/Makefile
@@ -124,7 +124,7 @@ $(BUILD)/firmware-combined.bin: $(BUILD)/firmware.elf
 
 $(BUILD)/firmware.elf: $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(LD) $(LDFLAGS) -o $@ $(OBJ) $(LIBS)
+	$(Q)$(LD) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 #MAKE_PINS = boards/make-pins.py
diff --git a/minimal/Makefile b/minimal/Makefile
index ddddebbe2..619d24952 100644
--- a/minimal/Makefile
+++ b/minimal/Makefile
@@ -59,7 +59,7 @@ all: $(BUILD)/firmware.elf
 
 $(BUILD)/firmware.elf: $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(LD) $(LDFLAGS) -o $@ $(OBJ) $(LIBS)
+	$(Q)$(LD) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 # Run emulation build on a POSIX system with suitable terminal settings
diff --git a/pic16bit/Makefile b/pic16bit/Makefile
index 98fcf07fd..309b47b79 100644
--- a/pic16bit/Makefile
+++ b/pic16bit/Makefile
@@ -57,7 +57,7 @@ $(BUILD)/firmware.hex: $(BUILD)/firmware.elf
 
 $(BUILD)/firmware.elf: $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(LD) $(LDFLAGS) -o $@ $(OBJ) $(LIBS)
+	$(Q)$(LD) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)size $@
 
 $(PY_BUILD)/gc.o: CFLAGS += -O1
diff --git a/py/mkrules.mk b/py/mkrules.mk
index 18bfd9847..26afdf227 100644
--- a/py/mkrules.mk
+++ b/py/mkrules.mk
@@ -81,14 +81,14 @@ all: $(PROG)
 
 $(PROG): $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(CC) $(COPT) -o $@ $(OBJ) $(LIB) $(LDFLAGS)
+	$(Q)$(CC) $(COPT) -o $@ $^ $(LIB) $(LDFLAGS)
 ifndef DEBUG
 	$(Q)$(STRIP) $(STRIPFLAGS_EXTRA) $(PROG)
 endif
 	$(Q)$(SIZE) $(PROG)
 
 lib: $(OBJ)
-	$(AR) rcs libmicropython.a $(OBJ)
+	$(AR) rcs libmicropython.a $^
 
 clean: clean-prog
 clean-prog:
diff --git a/qemu-arm/Makefile b/qemu-arm/Makefile
index a81dd67cf..31ba6baa2 100644
--- a/qemu-arm/Makefile
+++ b/qemu-arm/Makefile
@@ -74,11 +74,11 @@ $(BUILD)/tinytest.o:
 
 ## `$(LD)` doesn't seem to like `--specs` for some reason, but we can just use `$(CC)` here.
 $(BUILD)/firmware.elf: $(OBJ)
-	$(Q)$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $(OBJ) $(LIBS)
+	$(Q)$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 $(BUILD)/firmware-test.elf: $(OBJ_TEST)
-	$(Q)$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $(OBJ_TEST) $(LIBS)
+	$(Q)$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 include ../py/mkrules.mk
diff --git a/stmhal/Makefile b/stmhal/Makefile
index e50bf3b0f..222e19b32 100644
--- a/stmhal/Makefile
+++ b/stmhal/Makefile
@@ -285,7 +285,7 @@ $(BUILD)/firmware.hex: $(BUILD)/firmware.elf
 
 $(BUILD)/firmware.elf: $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(LD) $(LDFLAGS) -o $@ $(OBJ) $(LIBS)
+	$(Q)$(LD) $(LDFLAGS) -o $@ $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 MAKE_PINS = boards/make-pins.py
diff --git a/teensy/Makefile b/teensy/Makefile
index dece6ce00..32f753b3e 100644
--- a/teensy/Makefile
+++ b/teensy/Makefile
@@ -156,7 +156,7 @@ deploy: post_compile reboot
 
 $(BUILD)/micropython.elf: $(OBJ)
 	$(ECHO) "LINK $@"
-	$(Q)$(CC) $(LDFLAGS) -o "$@" -Wl,-Map,$(@:.elf=.map) $(OBJ) $(LIBS)
+	$(Q)$(CC) $(LDFLAGS) -o "$@" -Wl,-Map,$(@:.elf=.map) $^ $(LIBS)
 	$(Q)$(SIZE) $@
 
 ifeq ($(MEMZIP_DIR),)
-- 
GitLab