Skip to content

Commit 83d812e

Browse files
committed
Merge #73: ci: Use correct variable to designate C++ compiler
e051a7d ci: Install wine32 package for Windows tests (Hennadii Stepanov) 2d2c695 build: Drop unused `CC` variable (Hennadii Stepanov) 1810fcb ci: Use correct variable to designate C++ compiler (Hennadii Stepanov) Pull request description: Setting the `CC` variable does not affect the choice of a C++ compiler. `CXX` does. On the master branch (c36f1f0), CI tasks, which are supposed to compile using a non-default compiler (e.g., clang), actually are configured to use the default compiler. For example, https://api.cirrus-ci.com/v1/task/5042259930906624/logs/test.log: ``` + clang -v Debian clang version 11.0.1-2 Target: x86_64-pc-linux-gnu ... Build Options: with benchmarks = yes with tests = yes enable clmul fields = yes CXX = g++ -std=c++11 ... ``` ACKs for top commit: sipa: utACK e051a7d Tree-SHA512: 618c4071c105791437312283f7f06a4e4bec7029cae6b2cf385e7fe94a8e3c12441534be9f7ba22d1608b435efec4455c01e966981870541876ca94c7110f099
2 parents 022b959 + e051a7d commit 83d812e

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

.cirrus.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ task:
7272
<< : *ENV_MATRIX_SAN_VALGRIND
7373
matrix:
7474
- env:
75-
CC: gcc
75+
CXX: g++
7676
- env:
77-
CC: clang
77+
CXX: clang++ -gdwarf-4
7878
<< : *MERGE_BASE
7979
test_script:
8080
- ./ci/cirrus.sh
@@ -92,9 +92,10 @@ task:
9292
<< : *ENV_MATRIX_VALGRIND
9393
matrix:
9494
- env:
95-
CC: i686-linux-gnu-gcc
95+
CXX: i686-linux-gnu-g++
9696
- env:
97-
CC: clang --target=i686-pc-linux-gnu -isystem /usr/i686-linux-gnu/include
97+
CXX: clang++ --target=i686-linux-gnu -gdwarf-4
98+
CXXFLAGS: -g -O2 -isystem /usr/i686-linux-gnu/include -isystem /usr/i686-linux-gnu/include/c++/10/i686-linux-gnu
9899
test_script:
99100
- ./ci/cirrus.sh
100101
<< : *CAT_LOGS
@@ -110,9 +111,9 @@ task:
110111
<< : *ENV_MATRIX_SAN
111112
matrix:
112113
- env:
113-
CC: gcc-9
114+
CXX: g++-9
114115
- env:
115-
CC: clang
116+
CXX: clang++
116117
brew_script:
117118
- brew update
118119
- brew install automake libtool gcc@9

ci/cirrus.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export LC_ALL=C
77

88
env >> test_env.log
99

10-
$CC -v || true
10+
$CXX -v || true
1111
valgrind --version || true
1212

1313
./autogen.sh

ci/linux-debian.Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ RUN apt-get update
88
RUN apt-get install --no-install-recommends --no-upgrade -y \
99
git ca-certificates \
1010
make automake libtool pkg-config dpkg-dev valgrind qemu-user \
11-
gcc g++ clang libc6-dbg \
11+
gcc g++ clang libclang-rt-dev libc6-dbg \
1212
gcc-i686-linux-gnu g++-i686-linux-gnu libc6-dev-i386-cross libc6-dbg:i386 \
1313
g++-s390x-linux-gnu gcc-s390x-linux-gnu libc6-dev-s390x-cross libc6-dbg:s390x \
14-
wine g++-mingw-w64-x86-64
14+
wine wine32:i386 g++-mingw-w64-x86-64
1515

1616
# Run a dummy command in wine to make it set up configuration
1717
RUN wine true || true

configure.ac

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ if test "x$use_ccache" != "xno"; then
119119
fi
120120
else
121121
use_ccache=yes
122-
CC="$ac_cv_path_CCACHE $CC"
123122
CXX="$ac_cv_path_CCACHE $CXX"
124123
fi
125124
AC_MSG_RESULT($use_ccache)

0 commit comments

Comments
 (0)