This is an automated email from the ASF dual-hosted git repository.

raulcd 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 603901ab0a GH-47914: [C++] Fix system Apache ORC/Google logging used 
detection (#47915)
603901ab0a is described below

commit 603901ab0ae6740caabdb60bb16bfa8d451d959f
Author: Sutou Kouhei <[email protected]>
AuthorDate: Thu Oct 23 19:51:15 2025 +0900

    GH-47914: [C++] Fix system Apache ORC/Google logging used detection (#47915)
    
    ### Rationale for this change
    
    We need to use `orc_SOURCE` not `ORC_SOURCE` for Apache ORC and 
`glog_SOURCE` not `GLOG_SOURCE` for Google logging.
    
    ### What changes are included in this PR?
    
    Fix referred variable names.
    
    ### Are these changes tested?
    
    Yes in https://github.com/apache/arrow-java/pull/865 .
    
    ### Are there any user-facing changes?
    
    Yes.
    * GitHub Issue: #47914
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Raúl Cumplido <[email protected]>
---
 cpp/src/arrow/CMakeLists.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cpp/src/arrow/CMakeLists.txt b/cpp/src/arrow/CMakeLists.txt
index 73376774f3..25e5749335 100644
--- a/cpp/src/arrow/CMakeLists.txt
+++ b/cpp/src/arrow/CMakeLists.txt
@@ -80,13 +80,13 @@ if(ARROW_WITH_ZSTD)
 endif()
 
 if(ARROW_ORC)
-  if(ORC_SOURCE STREQUAL "SYSTEM")
+  if(orc_SOURCE STREQUAL "SYSTEM")
     list(APPEND ARROW_STATIC_INSTALL_INTERFACE_LIBS orc::orc)
   endif()
 endif()
 
 if(ARROW_USE_GLOG)
-  if(GLOG_SOURCE STREQUAL "SYSTEM")
+  if(glog_SOURCE STREQUAL "SYSTEM")
     list(APPEND ARROW_STATIC_INSTALL_INTERFACE_LIBS glog::glog)
   endif()
 endif()

Reply via email to