diff options
| author | Austin S. Hemmelgarn <austin@netdata.cloud> | 2024-02-22 07:08:44 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-02-22 07:08:44 -0500 |
| commit | a0d1e7ac696993af1da6f13ce021ced6f126717c (patch) | |
| tree | 8958d6a156ec6c070a073e03b27e7048db87f9c3 | |
| parent | 819025e9a755bec03f3d7e74152e9db16a008774 (diff) | |
Correctly mark protobuf as required in find_package. (#17021)
At the point at which we call find_package() to locate protobuf, we know
that we do, in fact, need it for this build. Given this, we should be
marking it as required in the find_package() call so that if it’s not
found the configuration phase fails instead of things breaking in
potentially strange ways later on during the build itself.
| -rw-r--r-- | CMakeLists.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index ef9f438587..7808f4312b 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -555,7 +555,7 @@ if(ENABLE_ACLK OR ENABLE_EXPORTER_PROMETHEUS_REMOTE_WRITE) set(Protobuf_USE_STATIC_LIBS On) endif() - find_package(Protobuf) + find_package(Protobuf REQUIRED) endif() set(ENABLE_PROTOBUF True) |