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 db1a8a9bf4 GH-47303: [C++] Don't install arrow-compute.pc twice 
(#47304)
db1a8a9bf4 is described below

commit db1a8a9bf42bec2b643fc19d8e174dd5166cd621
Author: Sutou Kouhei <[email protected]>
AuthorDate: Mon Aug 11 17:38:51 2025 +0900

    GH-47303: [C++] Don't install arrow-compute.pc twice (#47304)
    
    ### Rationale for this change
    
    Both of
    
    
https://github.com/apache/arrow/blob/97c9bfcdf8a9b864414fb5457a1c3f7a5747a3f1/cpp/src/arrow/CMakeLists.txt#L845-L866
    
    and
    
    
https://github.com/apache/arrow/blob/97c9bfcdf8a9b864414fb5457a1c3f7a5747a3f1/cpp/src/arrow/compute/CMakeLists.txt#L22-L25
    
    install `arrow-compute.pc`.
    
    We don't need to install `arrow-compute.pc` multiple times.
    
    ### What changes are included in this PR?
    
    Remove `arrow_add_pkg_config("arrow-compute")`.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    No.
    * GitHub Issue: #47303
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 cpp/src/arrow/compute/CMakeLists.txt | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/cpp/src/arrow/compute/CMakeLists.txt 
b/cpp/src/arrow/compute/CMakeLists.txt
index 6f740e86ee..28e30a564b 100644
--- a/cpp/src/arrow/compute/CMakeLists.txt
+++ b/cpp/src/arrow/compute/CMakeLists.txt
@@ -19,11 +19,6 @@ add_custom_target(arrow-compute-tests)
 
 arrow_install_all_headers("arrow/compute")
 
-if(ARROW_COMPUTE)
-  # pkg-config support
-  arrow_add_pkg_config("arrow-compute")
-endif()
-
 #
 # Unit tests
 #

Reply via email to