WIP

Author: hoffstadt-nasa

.github/workflows/build.yml

@@ -50,14 +50,14 @@ jobs:
         git clone https://github.com/PilotLightTech/pilotlight
         cd pilotlight-python
 
-    - name: Prep 1
+    - name: Build Python
       shell: cmd
       run: | 
         cd scripts
         python gen_build.py
         build_python_for_win32.bat
 
-    - name: Build Pilot Light Python
+    - name: Build Binaries
       shell: cmd
       run: |
         set VULKAN_SDK=D:/a/VulkanSDK
@@ -100,30 +100,60 @@ jobs:
         git clone https://github.com/PilotLightTech/pilotlight
         cd pilotlight-python
 
-    - name: Prep 1
+    - name: Build Python
       run: |
         cd $GITHUB_WORKSPACE
         cd scripts
         python3 gen_build.py
         chmod +x build_python_for_mac.sh
         ./build_python_for_mac.sh
 
-    - name: Build Pilot Light Python
+    - name: Build Binaries
       run: |
-        cd $GITHUB_WORKSPACE
-        cd src
+        set -Eeuo pipefail
+
+        # Build
+        pushd "$GITHUB_WORKSPACE/src" >/dev/null
         chmod +x build.sh
         ./build.sh
-        cd ..
-        cd out
-        ls
-        cd ../pilotlight
-        ls
-        cd ..
-        test -f ./out/pilotlight_python || exit 1
-        test -f ./out/libgltfd.a || exit 1
-        test -f ./out/libpl_platform_ext.a || exit 1
-        test -f ./pilotlight/pilotlight.so || exit 1
+        popd >/dev/null
+
+        echo "=== Directory snapshots for debugging ==="
+        echo "::group::ls -al ./out"
+        ls -al ./out || true
+        echo "::endgroup::"
+        echo "::group::ls -al ./pilotlight"
+        ls -al ./pilotlight || true
+        echo "::endgroup::"
+
+        # List of required files relative to repo root
+        required_files=(
+          "./out/pilotlight_python"
+          "./out/libglfwd.a"
+          "./out/libpl_platform_ext.a"
+          "./pilotlight/pilotlight.so"
+        )
+
+        # Check for missing files and collect all failures before exiting
+        missing=()
+        for f in "${required_files[@]}"; do
+          if [[ -f "$f" ]]; then
+            echo "✅ Found: $f"
+          else
+            echo "::error file=$f::Missing required file"
+            missing+=("$f")
+          fi
+        done
+
+        if ((${#missing[@]})); then
+          echo ""
+          echo "❌ Missing ${#missing[@]} required file(s):"
+          printf ' - %s\n' "${missing[@]}"
+          exit 1
+        else
+          echo "🎉 All required files are present."
+        fi
+
 
   Ubuntu:
 
@@ -168,27 +198,56 @@ jobs:
         git clone https://github.com/PilotLightTech/pilotlight
         cd pilotlight-python
 
-    - name: Prep 1
+    - name: Build Python
       run: |
         cd $GITHUB_WORKSPACE
         cd scripts
         python3 gen_build.py
         chmod +x build_python_for_linux.sh
         ./build_python_for_linux.sh
 
-    - name: Build Pilot Light Python
+    - name: Build Binaries
       run: |
-        cd $GITHUB_WORKSPACE
-        cd src
+        set -Eeuo pipefail
+
+        # Build
+        pushd "$GITHUB_WORKSPACE/src" >/dev/null
         chmod +x build.sh
         ./build.sh
-        cd ..
-        cd out
-        ls
-        cd ../pilotlight
-        ls
-        cd ..
-        test -f ./out/pilotlight_python || exit 1
-        test -f ./out/gltfd.a || exit 1
-        test -f ./out/pl_platform_ext.a || exit 1
-        test -f ./pilotlight/pilotlight.so || exit 1
+        popd >/dev/null
+
+        echo "=== Directory snapshots for debugging ==="
+        echo "::group::ls -al ./out"
+        ls -al ./out || true
+        echo "::endgroup::"
+        echo "::group::ls -al ./pilotlight"
+        ls -al ./pilotlight || true
+        echo "::endgroup::"
+
+        # List of required files relative to repo root
+        required_files=(
+          "./out/pilotlight_python"
+          "./out/glfwd.a"
+          "./out/pl_platform_ext.a"
+          "./pilotlight/pilotlight.so"
+        )
+
+        # Check for missing files and collect all failures before exiting
+        missing=()
+        for f in "${required_files[@]}"; do
+          if [[ -f "$f" ]]; then
+            echo "✅ Found: $f"
+          else
+            echo "::error file=$f::Missing required file"
+            missing+=("$f")
+          fi
+        done
+
+        if ((${#missing[@]})); then
+          echo ""
+          echo "❌ Missing ${#missing[@]} required file(s):"
+          printf ' - %s\n' "${missing[@]}"
+          exit 1
+        else
+          echo "🎉 All required files are present."
+        fi

.github/workflows/deploy.yml

@@ -55,7 +55,7 @@ jobs:
         git clone https://github.com/PilotLightTech/pilotlight
         cd pilotlight-python
 
-    - name: Build package
+    - name: Build Binaries
       shell: cmd
       run: |
         set VULKAN_SDK=D:/a/VulkanSDK
@@ -132,7 +132,7 @@ jobs:
         cd pilotlight-python
         python -m pip install pl-build
 
-    - name: Build Package
+    - name: Build Binaries
       run: |
         cd $GITHUB_WORKSPACE
         echo "$PL_PYTHON_INCLUDES"
@@ -196,7 +196,7 @@ jobs:
         cd pilotlight-python
         python -m pip install pl-build
 
-    - name: Build Package
+    - name: Build Binaries
       run: |
         cd $GITHUB_WORKSPACE
         echo "$PL_PYTHON_INCLUDES"