[PATCH] D143535: github: Add manual workflow to build and upload release binaries

Konrad Wilhelm Kleine via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 22 10:17:40 PST 2023


kwk added a comment.

Added some ideas. Maybe not all so good.



================
Comment at: .github/workflows/release-binaries.yml:12
+        required: true
+        default: 'true'
+      tag:
----------------



================
Comment at: .github/workflows/release-binaries.yml:15
+        description: 'Tag to build'
+        required: true
+
----------------



================
Comment at: .github/workflows/release-binaries.yml:15
+        description: 'Tag to build'
+        required: true
+
----------------
kwk wrote:
> 



================
Comment at: .github/workflows/release-binaries.yml:46
+      # | X.Y.Z     | -final
+      run: |
+        if [ -n "${{ inputs.tag }}" ]; then
----------------



================
Comment at: .github/workflows/release-binaries.yml:53
+
+        if [ -n "${{ inputs.upload }}" ]; then
+          upload="${{ inputs.upload }}"
----------------
I wonder if this will work. When I today tried this I got `false` and `true` values. That's why I check differently. The string comparison is safe this way.


================
Comment at: .github/workflows/set-release-binary-outputs.sh:25
+echo "rc-flags=$rc_flags" >> $GITHUB_OUTPUT
+echo "upload=$upload" >> $GITHUB_OUTPUT
+echo "ref=$tag" >> $GITHUB_OUTPUT
----------------
This one is not listed in the `outputs:` section of of the `prepare` job.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143535/new/

https://reviews.llvm.org/D143535



More information about the llvm-commits mailing list