diff options
author | Austin S. Hemmelgarn <austin@netdata.cloud> | 2024-03-20 07:13:44 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-20 07:13:44 -0400 |
commit | 804fd4ce548ba147f9b513517c3846cb6829f03d (patch) | |
tree | 4c66d0e1f9625f8ffea17a7a9527740f37ce7b4d /tests | |
parent | eb1b5478c200a402ebb6b395ab523045ac087187 (diff) |
Prefer Protobuf’s own CMake config over CMake's FindProtobuf. (#17128)
* Prefer Protobuf’s own CMake config over CMake's FindProtobuf.
The FindProtobuf CMake module shipped by upstream CMake is broken for
Protobuf version 22.0 and newer because it does not correctly pull in
the new Abseil dependencies. Protobuf itself sometimes ships a CMake
Package Configuration module that _does_ work correctly, so use that in
preference to the Find module shipped with CMake.
Upstream bug reference: https://gitlab.kitware.com/cmake/cmake/-/issues/24321
* Properly handle protoc executable.
* Restructure to explicitly handle fallback case ourselves.
This allows proper handling of compatibility code in a way that actually
works for us without us needing to ship a special module to handle the
compatibility case.
* Switch to bundling protobuf via CMake instead of an external script.
* Fix handling of Protobuf inclusion.
- Add correct include directories for protoc.
- Skip installing protobuf when installing the agent.
* Drop spurious quotation marks.
* Properly fix generator expression for protoc include paths.
* Properly default to bundling protobuf in installer code.
* Disable ASAN in unit tests.
It doesn’t work with the modified protobuf handling and per discussion
with the team is non-critical.
* Change comment based on review.
* Revert "Disable ASAN in unit tests."
This reverts commit 6cb98b1b59c55d639d68694424b98e790ba2f5a7.
* Disable IPMI and NFACCT plugins for unit tests.
We don’t actually have any unit tests for them, and they cause issues
building reliably in the unit testing environment.
* Disable ASAN for Abseil and Protobuf when vendoring them.
* Switch to commit hashes for protobuf/abseil.
* Restructure to better encapsulate protobuf handling as it’s own module.
* Fix up bundled protobuf version handling.
Google has complicated rules for C++ build environment support, so we
really need to be checking compiler versions and not _just_ C++ standard
version.
* Fix warnings about invalid defines.
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/run-unit-tests.sh | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/tests/run-unit-tests.sh b/tests/run-unit-tests.sh index 2d6e81ad39..f819f1bc95 100755 --- a/tests/run-unit-tests.sh +++ b/tests/run-unit-tests.sh @@ -26,8 +26,6 @@ install_netdata() { --install-prefix "$HOME" \ --dont-wait \ --dont-start-it \ - --enable-plugin-nfacct \ - --enable-plugin-freeipmi \ --disable-lto \ --enable-logsmanagement-tests } |