Skip to content

Commit 5d04144

Browse files
committed
[ci] Fix release artifacts build
Fix a bug where inputs are not provided to the release event of the release artifacts build. Signed-off-by: Schuyler Eldridge <[email protected]>
1 parent 4cb8914 commit 5d04144

File tree

1 file changed

+15
-6
lines changed

1 file changed

+15
-6
lines changed

.github/workflows/uploadReleaseArtifacts.yml

+15-6
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,10 @@ on:
3333
- relwithdebinfo
3434
- debug
3535
default: release
36+
manual_release:
37+
description: Trigger a manual full release build
38+
default: false
39+
type: boolean
3640

3741
# Run every day at 0700 UTC which is:
3842
# - 0000 PDT / 2300 PST
@@ -88,12 +92,17 @@ jobs:
8892
- name: Get Configuration
8993
id: get-config
9094
run: |
91-
config=$(.github/bin/uploadReleaseArtifacts.sh \
92-
-b ${{ inputs.cmake_build_type }} \
93-
-e ${{ github.event_name }} \
94-
-o ${{ inputs.os }} \
95-
${{ inputs.llvm_enable_assertions && '-a' || ' ' }} \
96-
${{ inputs.run_tests && '-t' || ' ' }})
95+
config=
96+
if [[ ${{ github.event_name }} == "release" || ${{ inputs.manual_release }} == "true" ]]; then
97+
config=$(.github/bin/uploadReleaseArtifacts.sh -e release)
98+
else
99+
config=$(.github/bin/uploadReleaseArtifacts.sh \
100+
-b ${{ inputs.cmake_build_type }} \
101+
-e ${{ github.event_name }} \
102+
-o ${{ inputs.os }} \
103+
${{ inputs.llvm_enable_assertions && '-a' || ' ' }} \
104+
${{ inputs.run_tests && '-t' || ' ' }})
105+
fi
97106
98107
echo "Build Matrix:" >> $GITHUB_STEP_SUMMARY
99108
echo "\`\`\`json" >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)