Skip to content

Commit f332884

Browse files
committed
Update CI setup to run a modern compiler in addition
In addition to out checks for older compilers (to support older systems), we should also use a new compiler to profit from extra checks and ensure we stay compatible to updated compiler versions.
1 parent e01fb1b commit f332884

File tree

4 files changed

+15
-9
lines changed

4 files changed

+15
-9
lines changed

.github/workflows/build.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,10 @@ jobs:
1414
matrix:
1515
build: [Debug, Release]
1616
llvm-major: [14]
17-
compiler: [clang++-14, clang++-15, g++-9]
17+
compiler: [clang++-14, clang++-19, g++-9]
1818
include:
1919
- llvm-major: 14
2020
llvm-minor: 0
21-
exclude:
22-
- llvm-major: 14
23-
compiler: clang++-15
2421

2522
continue-on-error: false
2623
steps:
@@ -53,9 +50,9 @@ jobs:
5350
run: |
5451
sudo apt-key adv --fetch-keys https://apt.llvm.org/llvm-snapshot.gpg.key
5552
sudo add-apt-repository -y 'deb http://apt.llvm.org/focal/ llvm-toolchain-focal-14 main'
56-
sudo add-apt-repository -y 'deb http://apt.llvm.org/focal/ llvm-toolchain-focal-15 main'
53+
sudo add-apt-repository -y 'deb http://apt.llvm.org/focal/ llvm-toolchain-focal-19 main'
5754
sudo apt-get update
58-
sudo apt-get -y install --no-install-recommends clang-${{ matrix.llvm-major }} llvm-${{ matrix.llvm-major }}-dev clang-tidy-14
55+
sudo apt-get -y install --no-install-recommends clang-${{ matrix.llvm-major }} llvm-${{ matrix.llvm-major }}-dev clang-tidy-19
5956
6057
- name: Workaround for sanitizer
6158
shell: bash
@@ -71,6 +68,9 @@ jobs:
7168
CXX: ${{ matrix.compiler }}
7269
shell: bash
7370
run: |
71+
echo $CXX
72+
$CXX -std=c++17 file.cpp
73+
ls
7474
mkdir build
7575
cd build
7676
cmake .. \

CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -119,11 +119,11 @@ endif()
119119

120120
find_program(
121121
VARA_FEATURE_RUN_CLANG_TIDY
122-
NAMES run-clang-tidy-14.py
123-
run-clang-tidy-14
122+
NAMES run-clang-tidy-19.py
123+
run-clang-tidy-19
124124
run-clang-tidy.py
125125
run-clang-tidy
126-
clang-tidy-14
126+
clang-tidy-19
127127
clang-tidy
128128
PATHS /usr/bin/ /usr/lib/llvm/*/share/clang/
129129
)

cmake/vara_feature_macros.cmake

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,8 @@ endif()
9797
function(check_std_filesystems varname)
9898
set(old_cmake_required_flags ${CMAKE_REQUIRED_FLAGS})
9999
set(CMAKE_REQUIRED_FLAGS "-std=c++17 ${CMAKE_REQUIRED_FLAGS}")
100+
set(CMAKE_REQUIRED_QUIET FALSE)
101+
message(STATUS "Args: ${CMAKE_REQUIRED_FLAGS}")
100102
check_cxx_source_compiles(
101103
"
102104
#include <filesystem>

file.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
2+
#include <filesystem>
3+
namespace fs = std::filesystem;
4+
int main() { return 0; }

0 commit comments

Comments
 (0)