diff options
| author | Egor Tensin <egor@tensin.name> | 2026-10-19 04:48:26 +0300 |
|---|---|---|
| committer | Egor Tensin <egor@tensin.name> | 2026-10-19 05:23:15 +0300 |
| commit | b1bfbd3501b6521d3204b5d060f88a8f6153f945 (patch) | |
| tree | bf2d3869023e1dd20bf395d5dca52f4b6b2b4f39 /.github/workflows/example_toolsets.yml | |
| parent | workflows/lint: add a zizmor job (diff) | |
| download | cmake-common-b1bfbd3501b6521d3204b5d060f88a8f6153f945.tar.gz cmake-common-b1bfbd3501b6521d3204b5d060f88a8f6153f945.zip | |
{actions,workflows}: fail on unset environment variables
The $env:NAME syntax fails to produce an error if $NAME is not defined,
regardless of Set-StrictMode/Set-PSDebug.
Also, while I'm at it, clean up the actions/workflows code a little bit.
Diffstat (limited to '.github/workflows/example_toolsets.yml')
| -rw-r--r-- | .github/workflows/example_toolsets.yml | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/.github/workflows/example_toolsets.yml b/.github/workflows/example_toolsets.yml index ad294d6..c3ecd90 100644 --- a/.github/workflows/example_toolsets.yml +++ b/.github/workflows/example_toolsets.yml @@ -76,15 +76,15 @@ jobs: with: persist-credentials: false - - name: Clean up PATH - uses: egor-tensin/cleanup-path@v5 - if: runner.os == 'Windows' - - name: Set common variables uses: $/.github/actions/common-variables with: toolset: ${{ matrix.toolset }} + - name: Clean up PATH + uses: egor-tensin/cleanup-path@v5 + if: env.CI_HOST_WINDOWS + - name: Set up Python uses: actions/setup-python@v7 with: @@ -111,9 +111,6 @@ jobs: - name: Set up software environment uses: $/.github/actions/software-environment - with: - toolset: ${{ matrix.toolset }} - platform: ${{ matrix.platform }} - name: Set up Visual Studio uses: egor-tensin/vs-shell@v2 @@ -125,7 +122,7 @@ jobs: id: build uses: $/.github/actions/build-example with: - src-dir: 'test/examples/${{ matrix.example }}' + src_dir: 'test/examples/${{ matrix.example }}' toolset: ${{ matrix.toolset }} platform: ${{ matrix.platform }} configuration: ${{ matrix.configuration }} @@ -133,22 +130,22 @@ jobs: - name: Verify runtime library linkage uses: $/.github/actions/check-runtime-library with: - path: ${{ steps.build.outputs.install-dir }} + install_dir: ${{ steps.build.outputs.install_dir }} - name: Verify architecture uses: $/.github/actions/check-arch with: - path: ${{ steps.build.outputs.install-dir }} + install_dir: ${{ steps.build.outputs.install_dir }} expected: ${{ matrix.platform }} - name: Run example project uses: $/.github/actions/run-example with: - path: ${{ steps.build.outputs.install-dir }} + install_dir: ${{ steps.build.outputs.install_dir }} #- name: Check symbols # uses: $/.github/actions/check-symbols # with: - # path: ${{ steps.build.outputs.install-dir }} + # install_dir: ${{ steps.build.outputs.install_dir }} # expected: ${{ toJson(matrix.symbols) }} # if: env.CI_TARGET_ELF |
