remove callable action and simplify build yml

Author: eeshanl

.github/actions/build-platform/action.yml

@@ -41,59 +41,41 @@ jobs:
           git config --global user.email "github-actions[bot]@users.noreply.github.com"
           git config --global --add safe.directory $GITHUB_WORKSPACE
 
-      - name: Stuart Build and Run 
-        uses: ./.github/actions/build-platform
-        with:
-          platform-config: 'Platforms/QemuSbsaPkg/PlatformBuild.py'
-          platform-name: 'SBSA'
-          tool-chain: 'GCC5'
-          stuart-args: 'HAF_TFA_BUILD=TRUE'
-          publish-logs: 'true'
+      - name: Stuart Setup
+        run: python Platforms/QemuSbsaPkg/PlatformBuild.py --setup
 
-      - name: Upload Firmware Binaries (artifacts)
-        uses: actions/upload-artifact@v4
-        with:
-          # One artifact per matrix target so they don't overwrite each other
-          name: haf-tfa-firmware
-          if-no-files-found: error
-          path: |
-            Build/QemuSbsaPkg/DEBUG_GCC5/HafTfaBins/**
-          retention-days: 14
+      - name: Stuart Update
+        run: python Platforms/QemuSbsaPkg/PlatformBuild.py --update
 
-  publish:
-    name: Publish HAF/TFA Firmware Binaries
-    needs: build
-    if: github.event_name == 'release'
-    runs-on: ubuntu-latest
+      - name: Stuart Build
+        run: python Platforms/QemuSbsaPkg/PlatformBuild.py HAF_TFA_BUILD=TRUE
 
-    permissions:
-      contents: write
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Download Artifacts
-        uses: actions/download-artifact@v4
-        with:
-          path: ${{ runner.temp }}/artifacts
+      - name: Install archive tools
+        if: github.event_name == 'release'
+        run: |
+          apt-get update -y
+          apt-get install -y zip tar gh
 
       - name: Create per-target archives
+        if: github.event_name == 'release'
+        shell: bash
         run: |
           set -euo pipefail
-          cd "${RUNNER_TEMP}/artifacts"
-          for d in */ ; do
-            d="${d%/}"
-            echo "Zipping $d ..."
-            ( cd "$d" && zip -r "${RUNNER_TEMP}/${d}-${{ github.event.release.tag_name }}.zip" . )
-            ( cd "$d" && tar -czf "${RUNNER_TEMP}/${d}-${{ github.event.release.tag_name }}.tar.gz" . )
-          done
-          echo "Done creating archives:"
-          ls -lh "${RUNNER_TEMP}"/*.zip "${RUNNER_TEMP}"/*.tar.gz
+          BIN_DIR="Build/QemuSbsaPkg/DEBUG_GCC5/HafTfaBins"
+          echo "Packaging binaries from ${BIN_DIR}"
+          if [ ! -d "$BIN_DIR" ]; then
+            echo "Binary directory not found: $BIN_DIR" >&2
+            exit 1
+          fi
+          ZIP_PATH="${RUNNER_TEMP}/haf-tfa-firmware-${{ github.event.release.tag_name }}.zip"
+          TAR_PATH="${RUNNER_TEMP}/haf-tfa-firmware-${{ github.event.release.tag_name }}.tar.gz"
+          (cd "$BIN_DIR" && zip -r "$ZIP_PATH" .)
+          (cd "$BIN_DIR" && tar -czf "$TAR_PATH" .)
+          echo "Created archives:"
+          ls -lh "$ZIP_PATH" "$TAR_PATH"
 
       - name: Upload Release Assets
+        if: github.event_name == 'release'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |

.github/workflows/build-haf-tfa.yml

@@ -316,16 +316,16 @@ def SetPlatformEnv(self):
         # Default turn on build reporting.
         self.env.SetValue("BUILDREPORTING", "TRUE", "Enabling build report")
         self.env.SetValue("BUILDREPORT_TYPES", "PCD DEPEX FLASH BUILD_FLAGS LIBRARY FIXED_ADDRESS HASH", "Setting build report types")
-        self.env.SetValue("ARM_TFA_PATH", os.path.join (self.GetWorkspaceRoot (), "Silicon/Arm/TFA"), "Platform hardcoded")
-        self.env.SetValue("ARM_HAF_PATH", os.path.join (self.GetWorkspaceRoot (), "Silicon/Arm/HAF"), "Platform hardcoded")
+        self.env.SetValue("ARM_TFA_PATH", str(Path(self.GetWorkspaceRoot()) / "Silicon" / "Arm" / "TFA"), "Platform hardcoded")
+        self.env.SetValue("ARM_HAF_PATH", str(Path(self.GetWorkspaceRoot()) / "Silicon" / "Arm" / "HAF"), "Platform hardcoded")
         self.env.SetValue("BLD_*_QEMU_CORE_NUM", "4", "Default")
         self.env.SetValue("BLD_*_MEMORY_PROTECTION", "TRUE", "Default")
         # Include the MFCI test cert by default, override on the commandline with "BLD_*_SHIP_MODE=TRUE" if you want the retail MFCI cert
         self.env.SetValue("BLD_*_SHIP_MODE", "FALSE", "Default")
         self.env.SetValue("CONF_AUTOGEN_INCLUDE_PATH", self.edk2path.GetAbsolutePathOnThisSystemFromEdk2RelativePath("QemuSbsaPkg", "Include"), "Platform Defined")
         self.env.SetValue("MU_SCHEMA_DIR", self.edk2path.GetAbsolutePathOnThisSystemFromEdk2RelativePath("QemuSbsaPkg", "CfgData"), "Platform Defined")
         self.env.SetValue("MU_SCHEMA_FILE_NAME", "QemuSbsaPkgCfgData.xml", "Platform Hardcoded")
-        self.env.SetValue("HAF_TFA_BUILD", "TRUE", "Default to false when running locally")
+        self.env.SetValue("HAF_TFA_BUILD", "TRUE", "Platform Hardcoded")
 
         if self.Helper.generate_secureboot_pcds(self) != 0:
             logging.error("Failed to generate include PCDs")