fix: Using setps.vars.output

gcarreno · Aug 31, 2023 · 3e337b0 · 3e337b0
1 parent b8dfff7
commit 3e337b0
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 11 deletions.
diff --git a/.github/workflows/build.lazarus.yml b/.github/workflows/build.lazarus.yml
@@ -23,13 +23,13 @@ on:
     outputs:
       win_artefact_path:
         description: Path to the stored artefact folder
-        value: ${{ jobs.build_app.outputs.win_artefact_path }}
+        value: ${{ steps.vars.outputs.win_artefact_path }}
       lin_artefact_path:
         description: Path to the stored artefact folder
-        value: ${{ jobs.build_app.outputs.lin_artefact_path }}
+        value: ${{ steps.vars.outputs.lin_artefact_path }}
       osx_artefact_path:
         description: Path to the stored artefact folder
-        value: ${{ jobs.build_app.outputs.lin_artefact_path }}
+        value: ${{ steps.vars.outputs.lin_artefact_path }}
 
 jobs:
 
@@ -52,10 +52,10 @@ jobs:
           - os: macos-latest
             triplet: ${{ fromJSON(inputs.triplets).osx }}
 
-    outputs:
-      win_artefact_path: ${{ steps.build_app.outputs.win_artefact_path }}
-      lin_artefact_path: ${{ steps.build_app.outputs.lin_artefact_path }}
-      osx_artefact_path: ${{ steps.build_app.outputs.osx_artefact_path }}
+#    outputs:
+#      win_artefact_path: ${{ steps.build_app.outputs.win_artefact_path }}
+#      lin_artefact_path: ${{ steps.build_app.outputs.lin_artefact_path }}
+#      osx_artefact_path: ${{ steps.build_app.outputs.osx_artefact_path }}
 
     steps:
       - name: Print some values
@@ -66,17 +66,26 @@ jobs:
           echo Matrix OS: ${{ matrix.os }}
           echo Executable: ${{ env._EXE_ }}
           echo Executable with triplet: ${{ env._EXE_TRIPLET_ }}
+
+      - name: Build outputs
+        run: |
           case "${{ matrix.os }}" in
             "windows-latest")
               echo Setting win_artefact_path to ${{ matrix.triplet }}
-              echo "win_artefact_path=${{ matrix.triplet }}" >> "${GITHUB_OUTPUT}"
+              echo echo ::set-output name=win_artefact_path::${{ matrix.triplet }}
             ;;
             "ubuntu-latest")
               echo Setting lin_artefact_path to ${{ matrix.triplet }}
-              echo "lin_artefact_path=${{ matrix.triplet }}" >> "${GITHUB_OUTPUT}"
+              echo echo ::set-output name=lin_artefact_path::${{ matrix.triplet }}
             ;;
             "macos-latest")
               echo Setting osx_artefact_path to ${{ matrix.triplet }}
-              echo "osx_artefact_path=${{ matrix.triplet }}" >> "${GITHUB_OUTPUT}"
+              echo echo ::set-output name=osx_artefact_path::${{ matrix.triplet }}
             ;;
           esac
+
+      - name: Print outputs
+        run: |
+          echo Win output: ${{ steps.vars.output.win_artefact_path }}
+          echo Lin output: ${{ steps.vars.output.lin_artefact_path }}
+          echo OSX output: ${{ steps.vars.output.osx_artefact_path }}
diff --git a/.github/workflows/main.lazarus.yml b/.github/workflows/main.lazarus.yml
@@ -46,7 +46,9 @@ jobs:
     needs: [ build, test ]
     with:
       artefacts: '[
-          "win": "${{ needs.build.outputs.win_artefact_path }}"
+          "win": "${{ needs.build.outputs.win_artefact_path }}",
+          "lin": "${{ needs.build.outputs.lin_artefact_path }}",
+          "osx": "${{ needs.build.outputs.osx_artefact_path }}"
         ]'
 
     strategy: