Fix build system regressions

This commit consolidates multiple build system fixes:
1. DTB compilation regression
   - Fixed unconditional DTB build in user-space mode
   - Fixed WebAssembly build missing DTB dependencies
   - Fixed DTB not being built on macOS-arm64 for SYSTEM mode
2. Makefile syntax error
   - Fixed TAB characters before $(warning) in mk/toolchain.mk
   - This prevented entire Makefile from parsing correctly
3. emcc configuration pollution
   - Fixed .config persistence causing ENABLE_SYSTEM=1 to leak
   - Added distclean before emcc builds to ensure clean state
4. Ubuntu ARM64 apt-get failures
   - Ignore non-critical dep11 (AppStream metadata) failures
   - Only retry on critical package index failures (Packages/Sources/Release)
   - Use || true for apt install to prevent exit code 100 from dep11 issues
   - Verify critical packages (make, git, curl, clang, bc) are installed
   - Exit with error only if actual package installation fails

Author: jserv

.github/workflows/main.yml

@@ -135,14 +135,15 @@ jobs:
     - name: default build using emcc
       if: success()
       run: |
+            make distclean
             make CC=emcc ENABLE_JIT=0 $PARALLEL
 
     - name: default build for system emulation using emcc
       if: success()
       run: |
             make distclean
             make CC=emcc ENABLE_SYSTEM=1 ENABLE_JIT=0 $PARALLEL
-            make distclean ENABLE_SYSTEM=1
+            make distclean
 
     - name: Build with various optimization levels
       if: success()
@@ -301,17 +302,50 @@ jobs:
         githubToken: ${{ github.token }}
         # No 'sudo' is available
         install: |
-          apt update -qq
-          apt install -yqq make git curl wget clang libsdl2-dev libsdl2-mixer-dev lsb-release software-properties-common gnupg bc
-          which wget || echo "WARNING: wget not found after installation"
+          # Retry apt update with exponential backoff for mirror sync issues
+          # Note: dep11 (AppStream metadata) failures are non-critical for build tools
+          for i in 1 2 3; do
+            apt update -qq --allow-releaseinfo-change 2>&1 | tee /tmp/apt-update.log
+            # Check for critical failures (package indices), ignore dep11 metadata
+            if ! grep -E "Failed to fetch.*/(Packages|Sources|Release)" /tmp/apt-update.log; then
+              echo "apt update succeeded (core package lists available)"
+              break
+            fi
+            if [ $i -lt 3 ]; then
+              delay=$((i * 30))
+              echo "apt update attempt $i: core package list errors, waiting ${delay}s..."
+              sleep $delay
+            else
+              echo "Warning: Proceeding after 3 attempts - dep11 metadata may be incomplete"
+            fi
+          done
+          # Install packages - exit 0 even if dep11 metadata is incomplete
+          apt install -yqq make git curl wget clang libsdl2-dev libsdl2-mixer-dev lsb-release software-properties-common gnupg bc 2>&1 | tee /tmp/apt-install.log || true
+          # Verify critical packages were installed
+          for pkg in make git curl clang bc; do
+            if ! command -v $pkg >/dev/null 2>&1; then
+              echo "ERROR: Critical package $pkg failed to install!"
+              cat /tmp/apt-install.log
+              exit 1
+            fi
+          done
+          echo "All critical build tools installed successfully"
         # FIXME: gcc build fails on Aarch64/Linux hosts
         env: |
           CC: clang-18
         # Append custom commands here
         run: |
           # Verify and install wget if needed (workaround for install step issues)
           if ! command -v wget > /dev/null 2>&1; then
-            apt update -qq && apt install -yqq wget
+            echo "wget not found, attempting to install..."
+            apt update -qq --allow-releaseinfo-change 2>&1 | tee /tmp/apt-update-wget.log || true
+            apt install -yqq wget 2>&1 | tee /tmp/wget-install.log || true
+            if ! command -v wget > /dev/null 2>&1; then
+              echo "ERROR: wget installation failed!"
+              cat /tmp/wget-install.log
+              exit 1
+            fi
+            echo "wget installed successfully"
           fi
           git config --global --add safe.directory ${{ github.workspace }}
           git config --global --add safe.directory ${{ github.workspace }}/src/softfloat
@@ -435,14 +469,15 @@ jobs:
      - name: default build using emcc
        if: success()
        run: |
+             make distclean
              make CC=emcc ENABLE_JIT=0 $PARALLEL
 
      - name: default build for system emulation using emcc
        if: success()
        run: |
              make distclean
              make CC=emcc ENABLE_SYSTEM=1 ENABLE_JIT=0 $PARALLEL
-             make distclean ENABLE_SYSTEM=1
+             make distclean
 
      - name: check + tests
        if: success()
@@ -499,14 +534,14 @@ jobs:
                fi
              done
 
-    - name: JIT debug test
-      env:
-        CC: ${{ steps.install_cc.outputs.cc }}
-      run: |
-            # Run JIT tests with debug mode to catch register allocation and cache coherency issues
-            make distclean && make ENABLE_JIT=1 ENABLE_JIT_DEBUG=1 check $PARALLEL
-            make distclean && make ENABLE_EXT_C=0 ENABLE_JIT=1 ENABLE_JIT_DEBUG=1 check $PARALLEL
-      if: ${{ always() }}
+     - name: JIT debug test
+       env:
+         CC: ${{ steps.install_cc.outputs.cc }}
+       run: |
+             # Run JIT tests with debug mode to catch register allocation and cache coherency issues
+             make distclean && make ENABLE_JIT=1 ENABLE_JIT_DEBUG=1 check $PARALLEL
+             make distclean && make ENABLE_EXT_C=0 ENABLE_JIT=1 ENABLE_JIT_DEBUG=1 check $PARALLEL
+       if: ${{ always() }}
 
      - name: undefined behavior test
        if: (success() || failure()) && steps.install_cc.outputs.cc == 'clang'  # gcc on macOS/arm64 does not support sanitizers

Makefile

@@ -392,7 +392,7 @@ DTB_DEPS := $(BUILD_DTB) $(BUILD_DTB2C)
 endif
 endif
 
-all: config $(DTB_DEPS) $(BUILD_DTB) $(BUILD_DTB2C) $(BIN)
+all: config $(DTB_DEPS) $(BIN)
 
 OBJS := \
     map.o \
@@ -437,7 +437,7 @@ $(OUT):
 
 $(BIN): $(OBJS) $(DEV_OBJS) | $(OUT)
 	$(VECHO) "  LD\t$@\n"
-	$(Q)$(CC) -o $@ $(CFLAGS_emcc) $^ $(LDFLAGS)
+	$(Q)$(CC) -o $@ $(CFLAGS_emcc) $(filter-out %.dtb %.h,$^) $(LDFLAGS)
 
 $(CONFIG_FILE): FORCE
 	$(Q)mkdir -p $(OUT)

mk/toolchain.mk

@@ -22,10 +22,10 @@ ifneq ($(shell $(CC) --version | head -n 1 | grep emcc),)
                 $(warning $(SDL_MUSIC_CANNOT_PLAY_WARNING))
             endif
         else
-	    $(warning $(SDL_MUSIC_CANNOT_PLAY_WARNING))
+        $(warning $(SDL_MUSIC_CANNOT_PLAY_WARNING))
         endif
     else
-	$(warning $(SDL_MUSIC_CANNOT_PLAY_WARNING))
+        $(warning $(SDL_MUSIC_CANNOT_PLAY_WARNING))
     endif
 
     # see commit 165c1a3 of emscripten

mk/wasm.mk

@@ -166,3 +166,21 @@ start-web: $(start_web_deps)
 .PHONY: check-demo-dir-exist start-web
 
 endif
+
+# For SYSTEM mode, DTB needs to be built regardless of whether we're using emcc
+# DTB is only built when SYSTEM=1 and ELF_LOADER=0
+ifeq ($(call has, SYSTEM), 1)
+ifeq ($(call has, ELF_LOADER), 0)
+# Add DTB as dependency for compilation stages
+# This is used by mk/system.mk for device object files
+deps_emcc += $(BUILD_DTB) $(BUILD_DTB2C)
+
+# For emcc builds: ensure DTB exists before emcc embeds it
+# Make BIN directly depend on DTB files as regular prerequisites
+# This will cause them to be built, but they'll also be passed to the linker
+# We need to filter them out in the linker command
+ifeq ("$(CC_IS_EMCC)", "1")
+$(BIN): $(BUILD_DTB) $(BUILD_DTB2C)
+endif
+endif
+endif