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 f10fadcedcd GH-55112: [Packaging][Debian] Add missing 
opentelemetry-cpp dependency (#55249)
f10fadcedcd is described below

commit f10fadcedcd9d02cad73a7ebd3d674bfe8e5d6bb
Author: Sutou Kouhei <[email protected]>
AuthorDate: Mon Oct 26 16:39:15 2026 +0900

    GH-55112: [Packaging][Debian] Add missing opentelemetry-cpp dependency 
(#55249)
    
    ### Rationale for this change
    
    `debuild` doesn't add OpenTelemetry dependencies to `libarrow.so.XXX` 
automatically because OpenTelemetry package on Debian doesn't have SO version.
    
    ### What changes are included in this PR?
    
    Add `opentelemetry-cpp` dependency explicitly.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    Yes.
    
    ### Was AI used for this PR?
    
    In accordance to the [AI generation 
guidelines](https://arrow.apache.org/docs/dev/developers/overview.html#ai-generated-code),
 please disclose below whether and how AI was used in this PR.
    
    **PR code and description written by:**
    
    - [x] Human
    - [ ] AI
    
    **Reviewed before submission by:**
    
    - [x] Human
    - [x] AI
    - [ ] Not reviewed
    
    * GitHub Issue: #55112
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Raúl Cumplido <[email protected]>
---
 dev/release/verify-apt.sh                           | 21 ++++++++++++++++++---
 .../linux-packages/apache-arrow/debian/control.in   |  3 ++-
 2 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/dev/release/verify-apt.sh b/dev/release/verify-apt.sh
index aa2d6ca1173..38051c9a43e 100755
--- a/dev/release/verify-apt.sh
+++ b/dev/release/verify-apt.sh
@@ -140,7 +140,24 @@ echo "::endgroup::"
 
 
 echo "::group::Test Apache Arrow C++"
+
+# 27.1.0-dev20261025 -> 27
+# 27.1.0 -> 27
+major_version=$(echo ${VERSION} | cut -d. -f1)
+# 27.1.0-dev20261025 -> 1
+# 27.1.0 -> 1
+minor_version=$(echo ${VERSION} | cut -d. -f2)
+# 27.1.0 -> 2701
+so_version=$((major_version * 100 + minor_version))
+
 mkdir -p build
+${APT_INSTALL} libarrow${so_version}=${package_version}
+libarrow_so="/usr/lib/$(uname -m)-linux-gnu/libarrow.so.${so_version}"
+# All dependencies must be installed.
+ldd "${libarrow_so}"
+if ldd "${libarrow_so}" | grep --fixed-strings 'not found'; then
+  exit 1
+fi
 ${APT_INSTALL} libarrow-dev=${package_version}
 required_packages=()
 required_packages+=(cmake)
@@ -258,9 +275,7 @@ elif [ -f /etc/apt/sources.list.d/apache-arrow.sources.bak 
]; then
   fi
 fi
 
-# 22.0.0.dev54 -> 22
-# 22.0.0 -> 22
-current_major_version=${package_version%%.*}
+current_major_version=${major_version}
 # 22 -> 21
 previous_major_version=$[current_major_version -1]
 previous_so_version="${previous_major_version}00"
diff --git a/dev/tasks/linux-packages/apache-arrow/debian/control.in 
b/dev/tasks/linux-packages/apache-arrow/debian/control.in
index acb57c1fbc1..a2d889c8acb 100644
--- a/dev/tasks/linux-packages/apache-arrow/debian/control.in
+++ b/dev/tasks/linux-packages/apache-arrow/debian/control.in
@@ -56,7 +56,8 @@ Multi-Arch: same
 Pre-Depends: ${misc:Pre-Depends}
 Depends:
   ${misc:Depends},
-  ${shlibs:Depends}
+  ${shlibs:Depends},
+@USE_SYSTEM_OPEN_TELEMETRY@  opentelemetry-cpp
 Description: Apache Arrow is a data processing library for analysis
  .
  This package provides C++ library files.

Reply via email to