Fix unit tests linking order + path in workflow

danielinux · danielinux · commit 99cbcab0a6bf · 2025-11-06T10:34:12.000+01:00
diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml
@@ -27,7 +27,7 @@ jobs:
 
       - name: Run unit tests
         run: |
-          ./build/unit
+          ./build/test/unit
 
       - name: Run TTL expired test
         run: |
diff --git a/Makefile b/Makefile
@@ -209,7 +209,7 @@ build/test/unit:
 	@echo "[CC] unit.c"
 	@$(CC) $(CFLAGS) $(UNIT_CFLAGS) -c src/test/unit/unit.c -o build/test/unit.o
 	@echo "[LD] $@"
-	@$(CC) $(UNIT_LDFLAGS) $(LDFLAGS) build/test/unit.o -o build/test/unit
+	@$(CC) build/test/unit.o -o build/test/unit $(UNIT_LDFLAGS) $(LDFLAGS)
 
 # Install dynamic library to re-link linux applications
 #

Original file line number	Diff line number	Diff line change
`@@ -209,7 +209,7 @@ build/test/unit:`
`209`	`209`	`@echo "[CC] unit.c"`
`210`	`210`	`@$(CC) $(CFLAGS) $(UNIT_CFLAGS) -c src/test/unit/unit.c -o build/test/unit.o`
`211`	`211`	`@echo "[LD] $@"`
`212`		`- @$(CC) $(UNIT_LDFLAGS) $(LDFLAGS) build/test/unit.o -o build/test/unit`
	`212`	`+ @$(CC) build/test/unit.o -o build/test/unit $(UNIT_LDFLAGS) $(LDFLAGS)`
`213`	`213`
`214`	`214`	`# Install dynamic library to re-link linux applications`
`215`	`215`	`#`