Fix link order in cosmopolitan.a

It turned out that the linker was doing the wrong with the amalgamation
library concerning weak stubs. A regression test has been added and new
binaries have been uploaded to https://justine.lol/cosmopolitan/

Ideally this should be fixed by building a tool that turns multiple .a
files into a single .a file with deduplication. As a workaround for now
the cosmopolitan.a build is restructured to not include LIBC_STUBS which
meant technical debt needed to be paid off where non-stub interfaces
were moved to LIBC_INTRIN and LIBC_NEXGEN32E.

Thank @PerfectProductions in #31 for the report!

Author: jart

Makefile

@@ -236,40 +236,44 @@ loc: o/$(MODE)/tool/build/summy.com
 	$(XARGS) wc -l | grep total | awk '{print $$1}' | $<
 
 COSMOPOLITAN_OBJECTS =		\
-	APE_LIB			\
-	LIBC			\
-	LIBC_ALG		\
-	LIBC_BITS		\
-	LIBC_CALLS		\
-	LIBC_CALLS_HEFTY	\
 	LIBC_CRYPTO		\
 	LIBC_DNS		\
-	LIBC_ELF		\
-	LIBC_FMT		\
-	LIBC_INTRIN		\
-	LIBC_LOG		\
-	LIBC_MEM		\
-	LIBC_NEXGEN32E		\
-	LIBC_NT			\
-	LIBC_OHMYPLUS		\
-	LIBC_RAND		\
-	LIBC_RUNTIME		\
 	LIBC_SOCK		\
-	LIBC_STDIO		\
-	LIBC_STR		\
-	LIBC_STUBS		\
-	LIBC_SYSV		\
-	LIBC_TIME		\
-	LIBC_TINYMATH		\
-	LIBC_UNICODE		\
+	LIBC_NT_WS2_32		\
+	LIBC_NT_MSWSOCK		\
+	LIBC_OHMYPLUS		\
 	LIBC_X			\
-	LIBC_ZIPOS		\
-	THIRD_PARTY_COMPILER_RT	\
-	THIRD_PARTY_DLMALLOC	\
 	THIRD_PARTY_GDTOA	\
 	THIRD_PARTY_GETOPT	\
+	LIBC_LOG		\
+	LIBC_UNICODE		\
+	LIBC_TIME		\
+	LIBC_ZIPOS		\
+	THIRD_PARTY_ZLIB	\
+	APE_LIB			\
 	THIRD_PARTY_MUSL	\
-	THIRD_PARTY_REGEX
+	LIBC_STDIO		\
+	LIBC_CALLS_HEFTY	\
+	THIRD_PARTY_REGEX	\
+	LIBC_ALG		\
+	LIBC_MEM		\
+	THIRD_PARTY_DLMALLOC	\
+	LIBC_BITS		\
+	LIBC_RUNTIME		\
+	LIBC_ELF		\
+	LIBC_CALLS		\
+	LIBC_RAND		\
+	LIBC_SYSV_CALLS		\
+	LIBC_NT_NTDLL		\
+	LIBC_NT_ADVAPI32	\
+	LIBC_FMT		\
+	THIRD_PARTY_COMPILER_RT	\
+	LIBC_TINYMATH		\
+	LIBC_SYSV		\
+	LIBC_STR		\
+	LIBC_INTRIN		\
+	LIBC_NT_KERNEL32	\
+	LIBC_NEXGEN32E
 
 COSMOPOLITAN_HEADERS =		\
 	LIBC			\
@@ -304,7 +308,9 @@ COSMOPOLITAN_HEADERS =		\
 	THIRD_PARTY_ZLIB	\
 	THIRD_PARTY_REGEX
 
-o/$(MODE)/cosmopolitan.a: $(filter-out o/libc/stubs/exit11.o,$(foreach x,$(COSMOPOLITAN_OBJECTS),$($(x)_OBJS)))
+o/$(MODE)/cosmopolitan.a.txt:
+	printf "%s\n" $(call reverse,$(call uniq,$(foreach x,$(COSMOPOLITAN_OBJECTS),$($(x)))))
+o/$(MODE)/cosmopolitan.a: $(filter-out o/libc/stubs/exit11.o,$(foreach x,$(COSMOPOLITAN_OBJECTS),$($(x)_A_OBJS)))
 o/cosmopolitan.h:				\
 		o/$(MODE)/tool/build/rollup.com	\
 		libc/integral/normalize.inc	\

ape/lib/apelib.mk

@@ -17,14 +17,10 @@ APE_LIB_A_SRCS =					\
 
 APE_LIB_A_OBJS =					\
 	$(APE_LIB_A_SRCS_S:%.S=o/$(MODE)/%.o)		\
-	$(APE_LIB_A_SRCS_C:%.c=o/$(MODE)/%.o)		\
-	$(APE_LIB_A_SRCS:%=o/$(MODE)/%.zip.o)		\
-	o/$(MODE)/ape/ape.lds.zip.o			\
-	o/$(MODE)/ape/ape.S.zip.o			\
-	o/$(MODE)/NOTICE.zip.o
+	$(APE_LIB_A_SRCS_C:%.c=o/$(MODE)/%.o)
 
 APE_LIB_A_CHECKS = $(APE_LIB_A_HDRS:%=o/$(MODE)/%.ok)
-APE_LIB_A_DIRECTDEPS = LIBC_STR LIBC_STUBS
+APE_LIB_A_DIRECTDEPS = LIBC_STR LIBC_NEXGEN32E LIBC_STUBS
 APE_LIB_A_DEPS = $(call uniq,$(foreach x,$(APE_LIB_A_DIRECTDEPS),$($(x))))
 
 $(APE_LIB_A): ape/lib/ $(APE_LIB_A).pkg $(APE_LIB_A_OBJS)

build/package

@@ -5,8 +5,8 @@
 if [ "$TOOL_BUILD_PACKAGE" ]; then
   set -- "$TOOL_BUILD_PACKAGE" "$@"
 else
-  if [ -x "o/tool/build/package.com" ]; then
-    set -- "o/tool/build/package.com" "$@"
+  if [ -x "o/tool/build/package.com.dbg" ]; then
+    set -- "o/tool/build/package.com.dbg" "$@"
   else
     MKDIR=${MKDIR:-$(command -v mkdir) -p} || exit
     CP=${CP:-$(command -v cp) -f} || exit

dsp/bmp/bmp.mk

@@ -16,7 +16,6 @@ DSP_BMP_A_SRCS =				\
 	$(DSP_BMP_A_SRCS_C)
 
 DSP_BMP_A_OBJS =				\
-	$(DSP_BMP_A_SRCS:%=o/$(MODE)/%.zip.o)	\
 	$(DSP_BMP_A_SRCS_S:%.S=o/$(MODE)/%.o)	\
 	$(DSP_BMP_A_SRCS_C:%.c=o/$(MODE)/%.o)
 

dsp/core/core.mk

@@ -16,7 +16,6 @@ DSP_CORE_A_SRCS =				\
 	$(DSP_CORE_A_SRCS_C)
 
 DSP_CORE_A_OBJS =				\
-	$(DSP_CORE_A_SRCS:%=o/$(MODE)/%.zip.o)	\
 	$(DSP_CORE_A_SRCS_S:%.S=o/$(MODE)/%.o)	\
 	$(DSP_CORE_A_SRCS_C:%.c=o/$(MODE)/%.o)
 

dsp/mpeg/mpeg.mk

@@ -16,7 +16,6 @@ DSP_MPEG_A_SRCS =				\
 	$(DSP_MPEG_A_SRCS_C)
 
 DSP_MPEG_A_OBJS =				\
-	$(DSP_MPEG_A_SRCS:%=o/$(MODE)/%.zip.o)	\
 	$(DSP_MPEG_A_SRCS_S:%.S=o/$(MODE)/%.o)	\
 	$(DSP_MPEG_A_SRCS_C:%.c=o/$(MODE)/%.o)
 
@@ -26,19 +25,20 @@ DSP_MPEG_A_CHECKS =				\
 
 DSP_MPEG_A_DIRECTDEPS =				\
 	LIBC_CALLS				\
+	LIBC_FMT				\
+	LIBC_INTRIN				\
 	LIBC_LOG				\
-	LIBC_RUNTIME				\
-	LIBC_TINYMATH				\
-	LIBC_TIME				\
-	LIBC_STUBS				\
-	LIBC_STR				\
+	LIBC_LOG				\
+	LIBC_MEM				\
 	LIBC_NEXGEN32E				\
+	LIBC_RUNTIME				\
 	LIBC_STDIO				\
+	LIBC_STR				\
+	LIBC_STUBS				\
 	LIBC_SYSV				\
-	LIBC_MEM				\
-	LIBC_LOG				\
-	LIBC_FMT				\
-	LIBC_UNICODE				\
+	LIBC_TIME				\
+	LIBC_TINYMATH				\
+	LIBC_UNICODE
 
 DSP_MPEG_A_DEPS :=				\
 	$(call uniq,$(foreach x,$(DSP_MPEG_A_DIRECTDEPS),$($(x))))

dsp/scale/scale.mk

@@ -16,7 +16,6 @@ DSP_SCALE_A_SRCS =				\
 	$(DSP_SCALE_A_SRCS_C)
 
 DSP_SCALE_A_OBJS =				\
-	$(DSP_SCALE_A_SRCS:%=o/$(MODE)/%.zip.o)	\
 	$(DSP_SCALE_A_SRCS_S:%.S=o/$(MODE)/%.o)	\
 	$(DSP_SCALE_A_SRCS_C:%.c=o/$(MODE)/%.o)
 
@@ -27,14 +26,15 @@ DSP_SCALE_A_CHECKS =				\
 DSP_SCALE_A_DIRECTDEPS =			\
 	DSP_CORE				\
 	LIBC_INTRIN				\
-	LIBC_NEXGEN32E				\
-	LIBC_TINYMATH				\
-	LIBC_TIME				\
-	LIBC_RUNTIME				\
 	LIBC_LOG				\
 	LIBC_MEM				\
-	LIBC_X					\
-	LIBC_STUBS
+	LIBC_NEXGEN32E				\
+	LIBC_RUNTIME				\
+	LIBC_STUBS				\
+	LIBC_STUBS				\
+	LIBC_TIME				\
+	LIBC_TINYMATH				\
+	LIBC_X
 
 DSP_SCALE_A_DEPS :=				\
 	$(call uniq,$(foreach x,$(DSP_SCALE_A_DIRECTDEPS),$($(x))))

dsp/tty/tty.mk

@@ -16,7 +16,6 @@ DSP_TTY_A_SRCS =				\
 	$(DSP_TTY_A_SRCS_C)
 
 DSP_TTY_A_OBJS =				\
-	$(DSP_TTY_A_SRCS:%=o/$(MODE)/%.zip.o)	\
 	$(DSP_TTY_A_SRCS_S:%.S=o/$(MODE)/%.o)	\
 	$(DSP_TTY_A_SRCS_C:%.c=o/$(MODE)/%.o)
 
@@ -29,6 +28,7 @@ DSP_TTY_A_DIRECTDEPS =				\
 	LIBC_ALG				\
 	LIBC_CALLS				\
 	LIBC_FMT				\
+	LIBC_INTRIN				\
 	LIBC_LOG				\
 	LIBC_RUNTIME				\
 	LIBC_MEM				\

examples/examples.mk

@@ -19,7 +19,6 @@ EXAMPLES_MAINS =						\
 	$(EXAMPLES_MAINS_CC)
 
 EXAMPLES_OBJS =							\
-	$(EXAMPLES_SRCS:%=o/$(MODE)/%.zip.o)			\
 	$(EXAMPLES_MAINS_S:%.S=o/$(MODE)/%.o)			\
 	$(EXAMPLES_MAINS_C:%.c=o/$(MODE)/%.o)			\
 	$(EXAMPLES_MAINS_CC:%.cc=o/$(MODE)/%.o)
@@ -47,6 +46,7 @@ EXAMPLES_DIRECTDEPS =						\
 	LIBC_CALLS						\
 	LIBC_CALLS_HEFTY					\
 	LIBC_FMT						\
+	LIBC_INTRIN						\
 	LIBC_LOG						\
 	LIBC_LOG_ASAN						\
 	LIBC_MEM						\

examples/package/build.mk

@@ -35,8 +35,7 @@ EXAMPLES_PACKAGE_BINS =					\
 # Remaps source file names to object names.
 # Also asks a wildcard rule to automatically run tool/build/zipobj.c
 EXAMPLES_PACKAGE_OBJS =					\
-	$(EXAMPLES_PACKAGE_SRCS:%.c=o/$(MODE)/%.o)	\
-	$(EXAMPLES_PACKAGE_SRCS:%=o/$(MODE)/%.zip.o)
+	$(EXAMPLES_PACKAGE_SRCS:%.c=o/$(MODE)/%.o)
 
 # Lists packages whose symbols are or may be directly referenced here.
 # Note that linking stubs is always a good idea due to synthetic code.