Skip to content

Commit

Permalink
unset CC only for mingw and arm32
Browse files Browse the repository at this point in the history
Since `scripts/test` always unset "CC", the github action for "clang"
in Linux does not use clang unintentionally.

This patch unset CC only for mingw{32,64} and arm32.
  • Loading branch information
nak3 committed Dec 17, 2024
1 parent f46d437 commit 95c74e2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@ jobs:
- os: "ubuntu-22.04"
arch: "native"
compiler: "clang"
- os: "ubuntu-24.04"
arch: "native"
compiler: "clang"
steps:
- name: "Checkout repository"
uses: actions/checkout@v4
Expand Down
5 changes: 4 additions & 1 deletion scripts/test
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,10 @@
set -e
set -x

unset CC
if [ "$ARCH" = "mingw32" ] || [ "$ARCH" = "mingw64" ] || [ "$ARCH" = "arm32" ] ; then
unset CC
fi

ENABLE_ASM="${ENABLE_ASM:=ON}"

if type apt-get >/dev/null 2>&1; then
Expand Down

0 comments on commit 95c74e2

Please sign in to comment.