Option to export Android Breakpad symbols (#26)

Author: hiimjasmine00

.github/workflows/test.yml

@@ -45,6 +45,7 @@ jobs:
           build-config: RelWithDebInfo
           export-pdb: true
           bundle-pdb: true
+          export-symbols: true
           path: textureldr
           combine: true
           target: ${{ matrix.config.target }}

README.md

@@ -38,6 +38,10 @@ https://github.com/geode-sdk/build-geode-mod/tree/main/examples
     # Default: false
     bundle-pdb: ''
 
+    # Whether to export Breakpad symbols for Android builds. Not required.
+    # Default: false
+    export-symbols: ''
+
     # Path to the project which to build. Defaults to current directory.
     path: ''
 

action.yml

@@ -66,6 +66,10 @@ inputs:
     description: Whether to bundle PDB files into the .geode package. Requires SDK v4.2.0 and CLI v3.3.0 at minimum. Defaults to false
     required: false
     default: false
+  export-symbols:
+    description: Whether to export Breakpad symbols for Android builds. Defaults to false
+    required: false
+    default: false
 
 outputs:
   build-output:
@@ -156,6 +160,15 @@ runs:
         curl -L https://github.com/ninja-build/ninja/releases/latest/download/ninja-${{ steps.platform.outputs.id }}.zip -o ninja.zip
         7z x ninja.zip -o"$GITHUB_WORKSPACE/ninja"
         echo "$GITHUB_WORKSPACE/ninja" >> $GITHUB_PATH
+
+    - name: Install dump_syms
+      if: inputs.export-symbols == 'true' && steps.platform.outputs.id == 'linux' && (steps.platform.outputs.target_id == 'android32' || steps.platform.outputs.target_id == 'android64')
+      shell: bash
+      run: |
+        curl -L https://github.com/mozilla/dump_syms/releases/latest/download/dump_syms-x86_64-unknown-linux-gnu.tar.xz -o dump_syms.tar.xz
+        xz -d dump_syms.tar.xz
+        tar -xf dump_syms.tar
+        echo "$GITHUB_WORKSPACE/dump_syms-x86_64-unknown-linux-gnu" >> $GITHUB_PATH
     
     # - name: Fix Ubuntu weirdness
     #   shell: bash
@@ -265,6 +278,14 @@ runs:
 
         for file in $(find ./build -name *.geode); do
           cp $file "${{ github.workspace }}/output"
+          if [ ${{ steps.platform.outputs.id }} = "linux" ] && [ "${{ inputs.export-symbols }}" = "true" ]; then
+            if [ ${{ steps.platform.outputs.target_id }} = "android32" ] || [ ${{ steps.platform.outputs.target_id }} = "android64" ]; then
+              SO_FILE_NAME="$(basename $file .geode).${{ steps.platform.outputs.target_id }}.so"
+              if [ -f "./build/$SO_FILE_NAME" ]; then
+                dump_syms ./build/$SO_FILE_NAME -o "${{ github.workspace }}/output/$SO_FILE_NAME.sym"
+              fi
+            fi
+          fi
         done
 
         if [ ${{ steps.platform.outputs.id }} = "win" ]; then
@@ -293,3 +314,4 @@ runs:
         path: |
           ${{ steps.build.outputs.output }}/*.geode
           ${{ steps.build.outputs.output }}/*.pdb
+          ${{ steps.build.outputs.output }}/*.sym

combine/action.yml

@@ -74,6 +74,9 @@ runs:
         done
         # Copy the .pdb file from windows artifact if it exists
         cp ./artifacts/geode-build-win/*.pdb out/ || true
+        # Copy the .sym files from android artifacts if they exist
+        cp ./artifacts/geode-build-android32/*.sym out/ || true
+        cp ./artifacts/geode-build-android64/*.sym out/ || true
         echo "output=$(realpath out)" >> $GITHUB_OUTPUT
 
     - uses: geekyeggo/delete-artifact@v5