You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ko...@apache.org on 2025/05/30 08:22:47 UTC
(arrow) branch main updated: MINOR: [C++] Fix an obvious opentelemetry CMake typo (#46644)
This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new f1c6619332 MINOR: [C++] Fix an obvious opentelemetry CMake typo (#46644)
f1c6619332 is described below
commit f1c6619332a280625a2381b03b2fd4a0722db82f
Author: Rossi Sun <za...@gmail.com>
AuthorDate: Fri May 30 01:22:42 2025 -0700
MINOR: [C++] Fix an obvious opentelemetry CMake typo (#46644)
### Rationale for this change
Obvious typo.
### What changes are included in this PR?
Fix a typo.
### Are these changes tested?
Yes.
### Are there any user-facing changes?
Yes.
Authored-by: Rossi Sun <za...@gmail.com>
Signed-off-by: Sutou Kouhei <ko...@clear-code.com>
---
cpp/cmake_modules/ThirdpartyToolchain.cmake | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index e218022370..29dde40716 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -4926,7 +4926,7 @@ macro(build_opentelemetry)
-DWITH_OTLP_HTTP_SSL_PREVIEW=OFF
-DWITH_OTLP_HTTP_SSL_TLS_PREVIEW=OFF
"-DProtobuf_INCLUDE_DIR=${OPENTELEMETRY_PROTOBUF_INCLUDE_DIR}"
- "-DProtobuf_LIBRARY=${OPENTELEMETRY_PROTOBUF_INCLUDE_DIR}"
+ "-DProtobuf_LIBRARY=${OPENTELEMETRY_PROTOBUF_LIBRARY}"
"-DProtobuf_PROTOC_EXECUTABLE=${OPENTELEMETRY_PROTOC_EXECUTABLE}")
# OpenTelemetry with OTLP enabled requires Protobuf definitions from a