diff options
| author | Kane York <kanepyork@gmail.com> | 2020-07-23T18·55-0700 |
|---|---|---|
| committer | kanepyork <rikingcoding@gmail.com> | 2020-07-23T21·46+0000 |
| commit | b8e94002d6a0a416a3128e1bc275d73f59700321 (patch) | |
| tree | 1a08833539e179abedde4590e37041c6aaeab2ee /third_party/nix/CMakeLists.txt | |
| parent | be98295dc24492f6607cce04ba586e3897a83543 (diff) | |
feat(3p/nix): run clang-tidy during builds r/1435
We need to -isystem the libcxx header files in order for clang-tidy to ignore them, as the Nix clang toolchain isn't doing that automatically. Change-Id: I05b9e9bd522de4c0e2ad543214f6bf6ab66a306b Reviewed-on: https://cl.tvl.fyi/c/depot/+/1359 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
Diffstat (limited to 'third_party/nix/CMakeLists.txt')
| -rw-r--r-- | third_party/nix/CMakeLists.txt | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/third_party/nix/CMakeLists.txt b/third_party/nix/CMakeLists.txt index 71651a9b1168..7591debb133e 100644 --- a/third_party/nix/CMakeLists.txt +++ b/third_party/nix/CMakeLists.txt @@ -7,6 +7,9 @@ set(CMAKE_CXX_STANDARD 17) # clangd and clang-tidy. set(CMAKE_EXPORT_COMPILE_COMMANDS ON) +# Enable warnings +set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Werror") + # Provide an output path for pkgconfig. include(GNUInstallDirs) set(PKGCONFIG_INSTALL_DIR ${CMAKE_INSTALL_LIBDIR}/pkgconfig) @@ -25,6 +28,16 @@ find_package(absl REQUIRED) find_package(gRPC REQUIRED) find_package(glog REQUIRED) +find_program(CLANG_TIDY_PATH clang-tidy) +if (CLANG_TIDY_PATH) + set(CMAKE_CXX_CLANG_TIDY "${CLANG_TIDY_PATH}") + + # nix's toolchain has a problem with system header includes, so clang-tidy requires a manual -isystem + if (DEFINED ENV{LIBCXX_INCLUDE}) + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -isystem $ENV{LIBCXX_INCLUDE}") + endif() +endif() + # generate a configuration file (autoheader-style) to configure # certain symbols that Nix depends on. configure_file(config.h.in nix_config.h @ONLY) |