From 4b0ca742d7f3ecbe397d5a05655538bd33567938 Mon Sep 17 00:00:00 2001 From: Szabolcs Nagy Date: Fri, 24 Jul 2020 14:37:31 +0000 Subject: [PATCH] Makefile: prevent linking duplicate object files previously the foo.OBJS list could not include foo.o because that's already linked into foo.exe, but this is an easy mistake to make when using foo.OBJS to link multiple objects into a test binary so ensure that objects are not duplicated on the link command line. --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 86b6ff5..cf2ab2b 100644 --- a/Makefile +++ b/Makefile @@ -133,11 +133,11 @@ $(B)/%.s:: src/%.c $(B)/%.lo:: src/%.c $(CC) $(CFLAGS) $($*.CFLAGS) -fPIC -DSHARED -c -o $@ $< 2>$@.err || echo BUILDERROR $@; cat $@.err $(B)/%.so: $(B)/%.lo - $(CC) -shared $(LDFLAGS) $($*.so.LDFLAGS) -o $@ $< $($*.so.LOBJS) $(LDLIBS) $($*.so.LDLIBS) 2>$@.err || echo BUILDERROR $@; cat $@.err + $(CC) -shared $(LDFLAGS) $($*.so.LDFLAGS) -o $@ $(sort $< $($*.so.LOBJS)) $(LDLIBS) $($*.so.LDLIBS) 2>$@.err || echo BUILDERROR $@; cat $@.err $(B)/%-static.exe: $(B)/%.o - $(CC) -static $(LDFLAGS) $($*-static.LDFLAGS) -o $@ $< $($*-static.OBJS) $(LDLIBS) $($*-static.LDLIBS) 2>$@.ld.err || echo BUILDERROR $@; cat $@.ld.err + $(CC) -static $(LDFLAGS) $($*-static.LDFLAGS) -o $@ $(sort $< $($*-static.OBJS)) $(LDLIBS) $($*-static.LDLIBS) 2>$@.ld.err || echo BUILDERROR $@; cat $@.ld.err $(B)/%.exe: $(B)/%.o - $(CC) $(LDFLAGS) $($*.LDFLAGS) -o $@ $< $($*.OBJS) $(LDLIBS) $($*.LDLIBS) 2>$@.ld.err || echo BUILDERROR $@; cat $@.ld.err + $(CC) $(LDFLAGS) $($*.LDFLAGS) -o $@ $(sort $< $($*.OBJS)) $(LDLIBS) $($*.LDLIBS) 2>$@.ld.err || echo BUILDERROR $@; cat $@.ld.err %.o.err: %.o touch $@ -- 2.20.1