Don't strip Go executables for Windows. (#2395)

toitlang · Jun 11, 2024 · b7f366f · b7f366f
1 parent 5b1448c
commit b7f366f
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -109,6 +109,12 @@ jobs:
           make BUILD=build-debug BUILD_TYPE=Debug sdk
           ccache -s
 
+      - name: Strip executables - Linux/Windows
+        if: runner.os != 'macOS' && (github.event_name == 'release' || inputs.strip == 'true')
+        # Note: Go executables may not be stripped on Windows.
+        run: |
+          strip -s build/host/sdk/bin/*
+
       # Test.
       - name: Test
         shell: bash  # This is crucial, as the powershell doesn't abort when there is an error.
@@ -209,12 +215,6 @@ jobs:
           echo "$TOITLSP" toitdoc -p $CPU_CORES --toitc=$TOITC "--sdk=$SDKDIR" "--out=$JSON_OUT_PATH" --version=$TOIT_VERSION lib
           "$TOITLSP" toitdoc -p $CPU_CORES --toitc=$TOITC "--sdk=$SDKDIR" "--out=$JSON_OUT_PATH" --version=$TOIT_VERSION lib
 
-      - name: Strip executables - Linux/Windows
-        if: runner.os != 'macOS' && (github.event_name == 'release' || inputs.strip == 'true')
-        run: |
-          strip -s build/host/sdk/bin/*
-          strip -s build/host/sdk/tools/*
-
       - name: Pack artifacts
         if: matrix.shard == 1
         shell: bash