Linking order matters

Makefile

@@ -2,18 +2,20 @@ FIRE_CXX ?= clang++
 FIRE_CXXFLAGS ?= -O3 -std=gnu++2a -Wall -Werror
 FIRE_LDLIBS ?= -lgflags -lglog -lpthread
 
-all: firecgi.a example_simple
+all: firecgi.a firecgi.o example_simple
 
-objects = firecgi.o connection.o request.o parse.o
+objects = server.o connection.o request.o parse.o
 
-_firebuf:
+firebuf/firebuf.o:
 	$(MAKE) --directory=firebuf
 
-firecgi.a: $(objects) _firebuf
-	ar x firebuf/firebuf.a
-	ar rcs $@ $(objects) $(shell ar t firebuf/firebuf.a)
+firecgi.a: $(objects)
+	ar rcs $@ $^
 
-example_simple: example_simple.o firecgi.a
+firecgi.o: $(objects) firebuf/firebuf.o
+	ld --relocatable --output=$@ $+
+
+example_simple: example_simple.o firecgi.o
 	$(FIRE_CXX) $(FIRE_CXXFLAGS) -o $@ $+ $(FIRE_LDLIBS)
 
 %.o: %.cc *.h Makefile
@@ -29,8 +31,7 @@ afl:
 
 afl_int: connection_afl
 
-connection_afl: connection_afl.o firecgi.a
-	$(MAKE) --directory=firebuf
+connection_afl: connection_afl.o firecgi.o
 	$(FIRE_CXX) $(FIRE_CXXFLAGS) -o $@ $+ $(FIRE_LDLIBS)
 
 test: test_connection

example_simple.cc

@@ -1,7 +1,7 @@
 #include <gflags/gflags.h>
 #include <glog/logging.h>
 
-#include "firecgi.h"
+#include "server.h"
 
 DEFINE_int32(port, 9000, "TCP port to bind");
 DEFINE_int32(threads, 1, "Number of server threads");

server.cc

@@ -6,7 +6,7 @@
 #include <sys/socket.h>
 #include <thread>
 
-#include "firecgi.h"
+#include "server.h"
 #include "connection.h"
 
 namespace firecgi {