Skip to content

Commit

Permalink
github: s/clang++-18/clang++/
Browse files Browse the repository at this point in the history
in 28b0555, we switched from ubuntu:jammy to fedora:40 as the
building environment of our CI workflow. and we replaced the
input parameter "clang++-18" in test.yml with "clang++", because
in fedora:40, the clang++ executable does not have the postfix
as the clang-18 installed by the setup-cpp action.

but we failed to update all of them in that change, that's why
the "check header" step was not performed after that change.

in this change, we replace that last "clang++-18" in the
workflow to "clang++".

Signed-off-by: Kefu Chai <kefu.chai@scylladb.com>
  • Loading branch information
tchaikov committed May 25, 2024
1 parent 343e601 commit 245ec8f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ jobs:
run: cmake --build build/${{inputs.mode}}

- name: Check Header
if: ${{ inputs.mode == 'dev' && inputs.compiler == 'clang++-18' }}
if: ${{ inputs.mode == 'dev' && inputs.compiler == 'clang++' }}
run: cmake --build build/${{ inputs.mode }} --target checkheaders

- name: Build with C++20 modules
Expand Down

0 comments on commit 245ec8f

Please sign in to comment.