Running phase: unpackPhase unpacking source archive /nix/store/bxwih2dxlpmglg1ppazbgqvnz69hibm3-source source root is source Running phase: patchPhase Running phase: updateAutotoolsGnuConfigScriptsPhase Running phase: configurePhase mesonConfigurePhase flags: --prefix=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4 --libdir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/lib --libexecdir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/libexec --bindir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/bin --sbindir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/sbin --includedir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/include --mandir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/share/man --infodir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/share/info --localedir=/nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/share/locale -Dauto_features=enabled -Dwrap_mode=nodownload --buildtype=plain -Db_pch=false The Meson build system Version: 1.10.2 Source dir: /nix/var/nix/builds/nix-50839-4061795475/source Build dir: /nix/var/nix/builds/nix-50839-4061795475/source/build Build type: native build WARNING: Project does not target a minimum version but uses feature introduced in '1.1': meson.options file. Use meson_options.txt instead Project name: nix-grpc-store Project version: 0.1.0-beta.4 C++ compiler for the host machine: clang++ (clang 22.1.8 "clang version 22.1.8") C++ linker for the host machine: clang++ ld64 956.6 Host machine cpu family: aarch64 Host machine cpu: aarch64 Found pkg-config: YES (/nix/store/kd2x1bj3gwfh2hb69fk5ym043h2qavxc-pkg-config-wrapper-0.29.2/bin/pkg-config) 0.29.2 Run-time dependency grpc++ found: YES 1.83.0 Run-time dependency protobuf found: YES 36.1.0 Run-time dependency libzstd found: YES 1.5.7 Run-time dependency threads found: YES Run-time dependency libcrypto found: YES 3.6.4 Run-time dependency prometheus-cpp-core found: YES 1.3.0 Run-time dependency prometheus-cpp-pull found: YES 1.3.0 Run-time dependency absl_flat_hash_map found: YES 20260107 Run-time dependency absl_hash found: YES 20260107 Run-time dependency absl_synchronization found: YES 20260107 Compiler for C++ supports arguments -Werror=thread-safety: YES Run-time dependency jemalloc found: YES 5.3.1_0 Run-time dependency nix-store found: YES 2.36.0pre20260920_5711f8fe Run-time dependency nix-util found: YES 2.36.0pre20260920_5711f8fe Run-time dependency libcurl found: YES 8.22.0 Run-time dependency nlohmann_json found: YES 3.12.0 Check usable header "jwt-cpp/traits/nlohmann-json/defaults.h" : YES Program protoc found: YES (/nix/store/r1npr0vx8d17c0fhh9gvg9mpaiphn62a-protobuf-36.1/bin/protoc) Program grpc_cpp_plugin found: YES (/nix/store/0mm81ldgq4ssz7gnh8589gx9zfcxpdhq-grpc-1.83.0/bin/grpc_cpp_plugin) Run-time dependency dl found: YES Program tests/farm-client-test.sh found: YES (/nix/var/nix/builds/nix-50839-4061795475/source/tests/farm-client-test.sh) Program tests/oidc-test.sh found: YES (/nix/var/nix/builds/nix-50839-4061795475/source/tests/oidc-test.sh) Build targets in project: 13 WARNING: Project specifies no minimum version but uses features which were added in versions: * 1.1: {'meson.options file'} nix-grpc-store 0.1.0-beta.4 User defined options auto_features: enabled b_pch : false bindir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/bin buildtype : plain includedir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/include infodir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/share/info libdir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/lib libexecdir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/libexec localedir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/share/locale mandir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/share/man prefix : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4 sbindir : /nix/store/0srgypjd6k2y5bzgpdhr78bmll4l0jpn-nix-grpc-store-clang-tidy-0.1.0-beta.4/sbin wrap_mode : nodownload Found ninja-1.13.2 at /nix/store/kalf64hrn0zrxlacfvgymwbck77qvsr9-ninja-1.13.2/bin/ninja mesonConfigurePhase: enabled\ parallel\ building Running phase: buildPhase [2/2] Generating nix_remote_proto with a custom command [0/1] /nix/store/j2s018h6gc2q0z3sw002br2zas3m1l9d-meson-1.10.2/bin/meson --internal clangtidy /nix/var/nix/builds/nix-50839-4061795475/source /nix/var/nix/builds/nix-50839-4061795475/source/build --color always >>> /nix/store/l8sbl3yky84xqxj3a5hazrjdx52avnf2-clang-tools-22.1.8/bin/clang-tidy --use-color -quiet -p /nix/var/nix/builds/nix-50839-4061795475/source/build /nix/var/nix/builds/nix-50839-4061795475/source/src/daemon/coordinator.cc Error while processing /nix/var/nix/builds/nix-50839-4061795475/source/src/daemon/coordinator.cc. ../src/daemon/log-spill.hh:38:27: error: parameter name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 38 | explicit UniqueFd(int fd) noexcept : fd_(fd) {} | ^ ../src/daemon/log-spill.hh:85:20: error: variable name 'at' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 85 | auto const at = written_.load(std::memory_order_relaxed); | ^ ../src/daemon/log-spill.hh:100:80: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:88: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:96: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:104: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:116:42: error: forwarding reference parameter 'fn' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 116 | auto drain(std::size_t offset, Fn && fn) const -> std::size_t | ^ ../src/daemon/log-spill.hh:116:42: error: parameter name 'fn' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] ../src/daemon/log-spill.hh:128:67: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 128 | auto const kind = std::to_integer(header[0]); | ^~~ ../src/daemon/log-spill.hh:149:25: error: variable name 'ec' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 149 | std::error_code ec; | ^ ../src/daemon/log-spill.hh:154:22: error: variable name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^ ../src/daemon/log-spill.hh:154:45: error: use of undeclared identifier 'O_TMPFILE' [clang-diagnostic-error] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^~~~~~~~~ ../src/daemon/log-spill.hh:154:77: error: 0600 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^ ../src/daemon/log-spill.hh:155:20: error: no viable conversion from returned value of type 'UniqueFd' to function return type 'std::expected' [clang-diagnostic-error] 155 | return fd; | ^~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:486:35: note: candidate constructor not viable: no known conversion from 'UniqueFd' to 'const expected &' for 1st argument 486 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected&) = delete; | ^ ~~~~~~~~~~~~~~~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:504:35: note: candidate constructor not viable: no known conversion from 'UniqueFd' to 'expected &&' for 1st argument 504 | _LIBCPP_HIDE_FROM_ABI constexpr expected(expected&& __other) noexcept( | ^ ~~~~~~~~~~~~~~~~~~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:546:7: note: candidate template ignored: could not match 'expected<_Up, _OtherErr>' against 'UniqueFd' 546 | expected(const expected<_Up, _OtherErr>& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:554:7: note: candidate template ignored: could not match 'expected<_Up, _OtherErr>' against 'UniqueFd' 554 | expected(expected<_Up, _OtherErr>&& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:563:7: note: candidate template ignored: constraints not satisfied [with _Up = UniqueFd &] 563 | expected(_Up&& __u) noexcept(is_nothrow_constructible_v<_Tp, _Up>) // strengthened | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:560:14: note: because 'is_constructible_v' evaluated to false 560 | is_constructible_v<_Tp, _Up> && !__is_std_unexpected>::value && | ^ ../src/daemon/log-spill.hh:158:18: error: variable name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 158 | UniqueFd fd(::mkostemp(name.data(), O_CLOEXEC)); | ^ ../src/daemon/log-spill.hh:169:24: error: variable name 'n' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 169 | auto const n = ::pwrite(fd_.get(), data.data(), data.size(), static_cast(offset)); | ^ ../src/daemon/log-spill.hh:185:24: error: variable name 'n' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 185 | auto const n = ::pread(fd_.get(), out.data(), out.size(), static_cast(offset)); | ^ ../src/daemon/log-spill.hh:212:31: error: parameter name 'cv' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 212 | std::condition_variable & cv, | ^ ../src/daemon/log-spill.hh:215:18: error: forwarding reference parameter 'cancelled' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 215 | Cancelled && cancelled, | ^ ../src/daemon/log-spill.hh:216:13: error: forwarding reference parameter 'emit' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 216 | Emit && emit) -> bool | ^ ../src/daemon/log-spill.hh:232:16: error: 'wait_for' should be placed inside a while statement or used with a conditional parameter [bugprone-spuriously-wake-up-functions,cert-con36-c,cert-con54-cpp,-warnings-as-errors] 232 | cv.wait_for(lock, poll); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:560:14: error: call to deleted constructor of 'nixgrpc::UniqueFd' [clang-diagnostic-error] 560 | is_constructible_v<_Tp, _Up> && !__is_std_unexpected>::value && | ^~~~~~~~~~~~~~~~~~ ../src/daemon/log-spill.hh:48:5: note: 'UniqueFd' has been explicitly marked deleted here 48 | UniqueFd(const UniqueFd &) = delete; | ^ ../src/daemon/log-spill.hh:34:7: note: 'UniqueFd' defined here 34 | class UniqueFd | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:568:87: note: candidate template ignored: could not match 'unexpected<_OtherErr>' against 'UniqueFd' 568 | _LIBCPP_HIDE_FROM_ABI constexpr explicit(!is_convertible_v) expected( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:575:7: note: candidate template ignored: could not match 'unexpected<_OtherErr>' against 'UniqueFd' 575 | expected(unexpected<_OtherErr>&& __unex) noexcept(is_nothrow_constructible_v<_Err, _OtherErr>) // strengthened | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:488:35: note: candidate constructor not viable: constraints not satisfied 488 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected&) | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:489:14: note: because 'is_copy_constructible_v' evaluated to false 489 | requires(is_copy_constructible_v<_Tp> && is_copy_constructible_v<_Err> && is_trivially_copy_constructible_v<_Tp> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:493:35: note: candidate constructor not viable: constraints not satisfied 493 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:495:14: note: because 'is_copy_constructible_v' evaluated to false 495 | requires(is_copy_constructible_v<_Tp> && is_copy_constructible_v<_Err> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:499:35: note: candidate constructor not viable: constraints not satisfied 499 | _LIBCPP_HIDE_FROM_ABI constexpr expected(expected&&) | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:500:79: note: because 'is_trivially_move_constructible_v' evaluated to false 500 | requires(is_move_constructible_v<_Tp> && is_move_constructible_v<_Err> && is_trivially_move_constructible_v<_Tp> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:580:44: note: explicit constructor is not a candidate 580 | _LIBCPP_HIDE_FROM_ABI constexpr explicit expected(in_place_t, _Args&&... __args) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:592:44: note: explicit constructor is not a candidate 592 | _LIBCPP_HIDE_FROM_ABI constexpr explicit expected(unexpect_t, _Args&&... __args) noexcept( | ^ >>> /nix/store/l8sbl3yky84xqxj3a5hazrjdx52avnf2-clang-tools-22.1.8/bin/clang-tidy --use-color -quiet -p /nix/var/nix/builds/nix-50839-4061795475/source/build /nix/var/nix/builds/nix-50839-4061795475/source/src/client/channel.cc [1/1] (1/2) Processing file /nix/var/nix/builds/nix-50839-4061795475/source/src/client/channel.cc. [1/1] (2/2) Processing file /nix/var/nix/builds/nix-50839-4061795475/source/src/client/channel.cc. >>> /nix/store/l8sbl3yky84xqxj3a5hazrjdx52avnf2-clang-tools-22.1.8/bin/clang-tidy --use-color -quiet -p /nix/var/nix/builds/nix-50839-4061795475/source/build /nix/var/nix/builds/nix-50839-4061795475/source/tests/log-spill-test.cc Error while processing /nix/var/nix/builds/nix-50839-4061795475/source/tests/log-spill-test.cc. ../src/daemon/log-spill.hh:38:27: error: parameter name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 38 | explicit UniqueFd(int fd) noexcept : fd_(fd) {} | ^ ../src/daemon/log-spill.hh:85:20: error: variable name 'at' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 85 | auto const at = written_.load(std::memory_order_relaxed); | ^ ../src/daemon/log-spill.hh:100:80: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:88: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:96: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:104: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:116:42: error: forwarding reference parameter 'fn' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 116 | auto drain(std::size_t offset, Fn && fn) const -> std::size_t | ^ ../src/daemon/log-spill.hh:116:42: error: parameter name 'fn' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] ../src/daemon/log-spill.hh:128:67: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 128 | auto const kind = std::to_integer(header[0]); | ^~~ ../src/daemon/log-spill.hh:149:25: error: variable name 'ec' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 149 | std::error_code ec; | ^ ../src/daemon/log-spill.hh:154:22: error: variable name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^ ../src/daemon/log-spill.hh:154:45: error: use of undeclared identifier 'O_TMPFILE' [clang-diagnostic-error] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^~~~~~~~~ ../src/daemon/log-spill.hh:154:77: error: 0600 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^ ../src/daemon/log-spill.hh:155:20: error: no viable conversion from returned value of type 'UniqueFd' to function return type 'std::expected' [clang-diagnostic-error] 155 | return fd; | ^~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:486:35: note: candidate constructor not viable: no known conversion from 'UniqueFd' to 'const expected &' for 1st argument 486 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected&) = delete; | ^ ~~~~~~~~~~~~~~~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:504:35: note: candidate constructor not viable: no known conversion from 'UniqueFd' to 'expected &&' for 1st argument 504 | _LIBCPP_HIDE_FROM_ABI constexpr expected(expected&& __other) noexcept( | ^ ~~~~~~~~~~~~~~~~~~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:546:7: note: candidate template ignored: could not match 'expected<_Up, _OtherErr>' against 'UniqueFd' 546 | expected(const expected<_Up, _OtherErr>& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:554:7: note: candidate template ignored: could not match 'expected<_Up, _OtherErr>' against 'UniqueFd' 554 | expected(expected<_Up, _OtherErr>&& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:563:7: note: candidate template ignored: constraints not satisfied [with _Up = UniqueFd &] 563 | expected(_Up&& __u) noexcept(is_nothrow_constructible_v<_Tp, _Up>) // strengthened | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:560:14: note: because 'is_constructible_v' evaluated to false 560 | is_constructible_v<_Tp, _Up> && !__is_std_unexpected>::value && | ^ ../src/daemon/log-spill.hh:158:18: error: variable name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 158 | UniqueFd fd(::mkostemp(name.data(), O_CLOEXEC)); | ^ ../src/daemon/log-spill.hh:169:24: error: variable name 'n' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 169 | auto const n = ::pwrite(fd_.get(), data.data(), data.size(), static_cast(offset)); | ^ ../src/daemon/log-spill.hh:185:24: error: variable name 'n' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 185 | auto const n = ::pread(fd_.get(), out.data(), out.size(), static_cast(offset)); | ^ ../src/daemon/log-spill.hh:212:31: error: parameter name 'cv' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 212 | std::condition_variable & cv, | ^ ../src/daemon/log-spill.hh:215:18: error: forwarding reference parameter 'cancelled' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 215 | Cancelled && cancelled, | ^ ../src/daemon/log-spill.hh:216:13: error: forwarding reference parameter 'emit' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 216 | Emit && emit) -> bool | ^ ../src/daemon/log-spill.hh:232:16: error: 'wait_for' should be placed inside a while statement or used with a conditional parameter [bugprone-spuriously-wake-up-functions,cert-con36-c,cert-con54-cpp,-warnings-as-errors] 232 | cv.wait_for(lock, poll); | ^ ../tests/log-spill-test.cc:1:1: error: included header atomic is not used directly [misc-include-cleaner,-warnings-as-errors] 1 | #include | ^~~~~~~~~~~~~~~~~ 2 | #include ../tests/log-spill-test.cc:23:45: error: no header providing "std::string_view" is directly included [misc-include-cleaner,-warnings-as-errors] 8 | #include 9 | #include 10 | #include 11 | 12 | #include "log-spill.hh" 13 | 14 | using nixgrpc::LogSpill; 15 | using nixgrpc::SpillKind; 16 | using Records = std::vector>; 17 | 18 | namespace { 19 | 20 | auto readAll(const LogSpill & spill) -> Records 21 | { 22 | Records out; 23 | spill.drain(0, [&](SpillKind kind, std::string_view text) -> void { out.emplace_back(kind, std::string(text)); }); | ^ ../tests/log-spill-test.cc:27:83: error: parameter name 'cv' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 27 | auto follow(const LogSpill & spill, std::mutex & mutex, std::condition_variable & cv, const bool & finished) -> Records | ^ ../tests/log-spill-test.cc:44:47: error: 300000 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 44 | spill.append(SpillKind::line, std::string(300000, 'x')); | ^ ../tests/log-spill-test.cc:47:15: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 47 | assert(got[0] == std::make_pair(SpillKind::line, std::string("first"))); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:48:15: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 48 | assert(got[1] == std::make_pair(SpillKind::phase, std::string("build"))); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:49:15: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 49 | assert(got[2].second.empty()); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:50:15: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 50 | assert(got[3].second.size() == 300000); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:57:36: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 57 | assert(rest.size() == 1 && rest[0].second == "later"); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:63:68: error: 5 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 63 | spill.append(SpillKind::line, std::string(LogSpill::kMaxText + 5, 'x')); | ^ ../tests/log-spill-test.cc:65:34: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 65 | assert(got.size() == 1 && got[0].second.size() == LogSpill::kMaxText); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:70:20: error: 64 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 70 | LogSpill spill(64); | ^ ../tests/log-spill-test.cc:71:25: error: 50 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 71 | for (int i = 0; i < 50; i++) { | ^ ../tests/log-spill-test.cc:85:36: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 85 | assert(only.size() == 1 && only[0].second == LogSpill::kTruncated); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:90:5: error: function is not thread safe [concurrency-mt-unsafe,-warnings-as-errors] 90 | setenv("TMPDIR", "/nonexistent-dir", 1); | ^ ../tests/log-spill-test.cc:90:5: error: no header providing "setenv" is directly included [misc-include-cleaner,-warnings-as-errors] 7 | setenv("TMPDIR", "/nonexistent-dir", 1); | ^ ../tests/log-spill-test.cc:95:5: error: function is not thread safe [concurrency-mt-unsafe,-warnings-as-errors] 95 | unsetenv("TMPDIR"); | ^ ../tests/log-spill-test.cc:95:5: error: no header providing "unsetenv" is directly included [misc-include-cleaner,-warnings-as-errors] ../tests/log-spill-test.cc:103:29: error: variable name 'cv' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 103 | std::condition_variable cv; | ^ ../tests/log-spill-test.cc:119:30: error: 1000 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 119 | while (spill.written() < 1000) { | ^ ../tests/log-spill-test.cc:129:26: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 129 | assert((*got)[static_cast(i)].second == "line " + std::to_string(i)); | ^ /nix/store/rq88pmfxfl98jhzfhb46jvl0rl8nyd6g-apple-sdk-14.4/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/assert.h:99:25: note: expanded from macro 'assert' 99 | (__builtin_expect(!(e), 0) ? __assert_rtn(__func__, __ASSERT_FILE_NAME, __LINE__, #e) : (void)0) | ^ ../tests/log-spill-test.cc:138:29: error: variable name 'cv' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 138 | std::condition_variable cv; | ^ ../tests/log-spill-test.cc:149:6: error: an exception may be thrown in function 'main' which should not throw exceptions [bugprone-exception-escape,-warnings-as-errors] 149 | auto main() -> int | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__new/exceptions.h:67:3: note: frame #0: unhandled exception of type 'bad_array_new_length' may be thrown in function '__throw_bad_array_new_length' here 67 | throw bad_array_new_length(); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__memory/allocator.h:101:7: note: frame #1: function 'allocate' calls function '__throw_bad_array_new_length' here 101 | std::__throw_bad_array_new_length(); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__memory/allocator.h:112:13: note: frame #2: function 'allocate_at_least' calls function 'allocate' here 112 | return {allocate(__n), __n}; | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__memory/allocator_traits.h:280:14: note: frame #3: function 'allocate_at_least>' calls function 'allocate_at_least' here 280 | return __alloc.allocate_at_least(__n); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__memory/allocate_at_least.h:26:10: note: frame #4: function '__allocate_at_least>' calls function 'allocate_at_least>' here 26 | return std::allocator_traits<_Alloc>::allocate_at_least(__alloc, __n); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/string:2749:23: note: frame #5: function '__grow_by' calls function '__allocate_at_least>' here 2749 | auto __allocation = std::__allocate_at_least(__alloc_, __cap + 1); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/string:2776:3: note: frame #6: function '__grow_by_without_replace' calls function '__grow_by' here 2776 | __grow_by(__old_cap, __delta_cap, __old_sz, __n_copy, __n_del, __n_add); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/string:3041:7: note: frame #7: function 'append' calls function '__grow_by_without_replace' here 3041 | __grow_by_without_replace(__cap, __sz + __n - __cap, __sz, __sz, 0); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/string:3394:5: note: frame #8: function 'resize' calls function 'append' here 3394 | append(__n - __sz, __c); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/string:1307:84: note: frame #9: function 'resize' calls function 'resize' here 1307 | _LIBCPP_HIDE_FROM_ABI _LIBCPP_CONSTEXPR_SINCE_CXX20 void resize(size_type __n) { resize(__n, value_type()); } | ^ ../src/daemon/log-spill.hh:132:13: note: frame #10: function 'drain<(lambda at ../tests/log-spill-test.cc:23:20)>' calls function 'resize' here 132 | text.resize(len); | ^ ../tests/log-spill-test.cc:23:5: note: frame #11: function 'readAll' calls function 'drain<(lambda at ../tests/log-spill-test.cc:23:20)>' here 23 | spill.drain(0, [&](SpillKind kind, std::string_view text) -> void { out.emplace_back(kind, std::string(text)); }); | ^ ../tests/log-spill-test.cc:40:12: note: frame #12: function 'roundTrip' calls function 'readAll' here 40 | assert(readAll(spill).empty()); | ^ ../tests/log-spill-test.cc:151:5: note: frame #13: function 'main' calls function 'roundTrip' here 151 | roundTrip(); | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:560:14: error: call to deleted constructor of 'nixgrpc::UniqueFd' [clang-diagnostic-error] 560 | is_constructible_v<_Tp, _Up> && !__is_std_unexpected>::value && | ^~~~~~~~~~~~~~~~~~ ../src/daemon/log-spill.hh:48:5: note: 'UniqueFd' has been explicitly marked deleted here 48 | UniqueFd(const UniqueFd &) = delete; | ^ ../src/daemon/log-spill.hh:34:7: note: 'UniqueFd' defined here 34 | class UniqueFd | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:568:87: note: candidate template ignored: could not match 'unexpected<_OtherErr>' against 'UniqueFd' 568 | _LIBCPP_HIDE_FROM_ABI constexpr explicit(!is_convertible_v) expected( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:575:7: note: candidate template ignored: could not match 'unexpected<_OtherErr>' against 'UniqueFd' 575 | expected(unexpected<_OtherErr>&& __unex) noexcept(is_nothrow_constructible_v<_Err, _OtherErr>) // strengthened | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:488:35: note: candidate constructor not viable: constraints not satisfied 488 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected&) | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:489:14: note: because 'is_copy_constructible_v' evaluated to false 489 | requires(is_copy_constructible_v<_Tp> && is_copy_constructible_v<_Err> && is_trivially_copy_constructible_v<_Tp> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:493:35: note: candidate constructor not viable: constraints not satisfied 493 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:495:14: note: because 'is_copy_constructible_v' evaluated to false 495 | requires(is_copy_constructible_v<_Tp> && is_copy_constructible_v<_Err> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:499:35: note: candidate constructor not viable: constraints not satisfied 499 | _LIBCPP_HIDE_FROM_ABI constexpr expected(expected&&) | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:500:79: note: because 'is_trivially_move_constructible_v' evaluated to false 500 | requires(is_move_constructible_v<_Tp> && is_move_constructible_v<_Err> && is_trivially_move_constructible_v<_Tp> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:580:44: note: explicit constructor is not a candidate 580 | _LIBCPP_HIDE_FROM_ABI constexpr explicit expected(in_place_t, _Args&&... __args) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:592:44: note: explicit constructor is not a candidate 592 | _LIBCPP_HIDE_FROM_ABI constexpr explicit expected(unexpect_t, _Args&&... __args) noexcept( | ^ >>> /nix/store/l8sbl3yky84xqxj3a5hazrjdx52avnf2-clang-tools-22.1.8/bin/clang-tidy --use-color -quiet -p /nix/var/nix/builds/nix-50839-4061795475/source/build /nix/var/nix/builds/nix-50839-4061795475/source/src/daemon/main.cc Errors encountered while running clang-tidy Error while processing /nix/var/nix/builds/nix-50839-4061795475/source/src/daemon/main.cc. ../src/daemon/log-spill.hh:38:27: error: parameter name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 38 | explicit UniqueFd(int fd) noexcept : fd_(fd) {} | ^ ../src/daemon/log-spill.hh:85:20: error: variable name 'at' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 85 | auto const at = written_.load(std::memory_order_relaxed); | ^ ../src/daemon/log-spill.hh:100:80: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:88: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:96: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:100:104: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 100 | std::array header{static_cast(kind), len[0], len[1], len[2], len[3]}; | ^~~ ../src/daemon/log-spill.hh:116:42: error: forwarding reference parameter 'fn' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 116 | auto drain(std::size_t offset, Fn && fn) const -> std::size_t | ^ ../src/daemon/log-spill.hh:116:42: error: parameter name 'fn' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] ../src/daemon/log-spill.hh:128:67: error: possibly unsafe 'operator[]', consider bounds-safe alternatives [cppcoreguidelines-pro-bounds-avoid-unchecked-container-access,-warnings-as-errors] 128 | auto const kind = std::to_integer(header[0]); | ^~~ ../src/daemon/log-spill.hh:149:25: error: variable name 'ec' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 149 | std::error_code ec; | ^ ../src/daemon/log-spill.hh:154:22: error: variable name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^ ../src/daemon/log-spill.hh:154:45: error: use of undeclared identifier 'O_TMPFILE' [clang-diagnostic-error] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^~~~~~~~~ ../src/daemon/log-spill.hh:154:77: error: 0600 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers,-warnings-as-errors] 154 | if (UniqueFd fd(::open(dir.c_str(), O_TMPFILE | O_RDWR | O_CLOEXEC, 0600)); fd) { | ^ ../src/daemon/log-spill.hh:155:20: error: no viable conversion from returned value of type 'UniqueFd' to function return type 'std::expected' [clang-diagnostic-error] 155 | return fd; | ^~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:486:35: note: candidate constructor not viable: no known conversion from 'UniqueFd' to 'const expected &' for 1st argument 486 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected&) = delete; | ^ ~~~~~~~~~~~~~~~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:504:35: note: candidate constructor not viable: no known conversion from 'UniqueFd' to 'expected &&' for 1st argument 504 | _LIBCPP_HIDE_FROM_ABI constexpr expected(expected&& __other) noexcept( | ^ ~~~~~~~~~~~~~~~~~~ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:546:7: note: candidate template ignored: could not match 'expected<_Up, _OtherErr>' against 'UniqueFd' 546 | expected(const expected<_Up, _OtherErr>& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:554:7: note: candidate template ignored: could not match 'expected<_Up, _OtherErr>' against 'UniqueFd' 554 | expected(expected<_Up, _OtherErr>&& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:563:7: note: candidate template ignored: constraints not satisfied [with _Up = UniqueFd &] 563 | expected(_Up&& __u) noexcept(is_nothrow_constructible_v<_Tp, _Up>) // strengthened | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:560:14: note: because 'is_constructible_v' evaluated to false 560 | is_constructible_v<_Tp, _Up> && !__is_std_unexpected>::value && | ^ ../src/daemon/log-spill.hh:158:18: error: variable name 'fd' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 158 | UniqueFd fd(::mkostemp(name.data(), O_CLOEXEC)); | ^ ../src/daemon/log-spill.hh:169:24: error: variable name 'n' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 169 | auto const n = ::pwrite(fd_.get(), data.data(), data.size(), static_cast(offset)); | ^ ../src/daemon/log-spill.hh:185:24: error: variable name 'n' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 185 | auto const n = ::pread(fd_.get(), out.data(), out.size(), static_cast(offset)); | ^ ../src/daemon/log-spill.hh:212:31: error: parameter name 'cv' is too short, expected at least 3 characters [readability-identifier-length,-warnings-as-errors] 212 | std::condition_variable & cv, | ^ ../src/daemon/log-spill.hh:215:18: error: forwarding reference parameter 'cancelled' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 215 | Cancelled && cancelled, | ^ ../src/daemon/log-spill.hh:216:13: error: forwarding reference parameter 'emit' is never forwarded inside the function body [cppcoreguidelines-missing-std-forward,-warnings-as-errors] 216 | Emit && emit) -> bool | ^ ../src/daemon/log-spill.hh:232:16: error: 'wait_for' should be placed inside a while statement or used with a conditional parameter [bugprone-spuriously-wake-up-functions,cert-con36-c,cert-con54-cpp,-warnings-as-errors] 232 | cv.wait_for(lock, poll); | ^ ../src/daemon/main.cc:96:41: error: no header providing "nixgrpc::SpillKind" is directly included [misc-include-cleaner,-warnings-as-errors] 71 | static_assert(static_cast(nixgrpc::SpillKind::line) == static_cast(nixgrpc::BuildEvent::Kind::line)); | ^ ../src/daemon/main.cc:507:41: error: no header providing "nixgrpc::followSpill" is directly included [misc-include-cleaner,-warnings-as-errors] 507 | auto const completed = nixgrpc::followSpill( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:560:14: error: call to deleted constructor of 'nixgrpc::UniqueFd' [clang-diagnostic-error] 560 | is_constructible_v<_Tp, _Up> && !__is_std_unexpected>::value && | ^~~~~~~~~~~~~~~~~~ ../src/daemon/log-spill.hh:48:5: note: 'UniqueFd' has been explicitly marked deleted here 48 | UniqueFd(const UniqueFd &) = delete; | ^ ../src/daemon/log-spill.hh:34:7: note: 'UniqueFd' defined here 34 | class UniqueFd | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:568:87: note: candidate template ignored: could not match 'unexpected<_OtherErr>' against 'UniqueFd' 568 | _LIBCPP_HIDE_FROM_ABI constexpr explicit(!is_convertible_v) expected( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:575:7: note: candidate template ignored: could not match 'unexpected<_OtherErr>' against 'UniqueFd' 575 | expected(unexpected<_OtherErr>&& __unex) noexcept(is_nothrow_constructible_v<_Err, _OtherErr>) // strengthened | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:488:35: note: candidate constructor not viable: constraints not satisfied 488 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected&) | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:489:14: note: because 'is_copy_constructible_v' evaluated to false 489 | requires(is_copy_constructible_v<_Tp> && is_copy_constructible_v<_Err> && is_trivially_copy_constructible_v<_Tp> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:493:35: note: candidate constructor not viable: constraints not satisfied 493 | _LIBCPP_HIDE_FROM_ABI constexpr expected(const expected& __other) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:495:14: note: because 'is_copy_constructible_v' evaluated to false 495 | requires(is_copy_constructible_v<_Tp> && is_copy_constructible_v<_Err> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:499:35: note: candidate constructor not viable: constraints not satisfied 499 | _LIBCPP_HIDE_FROM_ABI constexpr expected(expected&&) | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:500:79: note: because 'is_trivially_move_constructible_v' evaluated to false 500 | requires(is_move_constructible_v<_Tp> && is_move_constructible_v<_Err> && is_trivially_move_constructible_v<_Tp> && | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:580:44: note: explicit constructor is not a candidate 580 | _LIBCPP_HIDE_FROM_ABI constexpr explicit expected(in_place_t, _Args&&... __args) noexcept( | ^ /nix/store/9z187ss3liak3pngakax2ip7nb1dc7z5-libcxx-21.1.6+apple-sdk-26.5/include/c++/v1/__expected/expected.h:592:44: note: explicit constructor is not a candidate 592 | _LIBCPP_HIDE_FROM_ABI constexpr explicit expected(unexpect_t, _Args&&... __args) noexcept( | ^ FAILED: [code=1] meson-internal__clang-tidy /nix/store/j2s018h6gc2q0z3sw002br2zas3m1l9d-meson-1.10.2/bin/meson --internal clangtidy /nix/var/nix/builds/nix-50839-4061795475/source /nix/var/nix/builds/nix-50839-4061795475/source/build --color always ninja: build stopped: subcommand failed.