You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ks...@apache.org on 2022/06/08 07:51:13 UTC
[arrow] branch master updated: ARROW-16785: [Packaging][Linux] Add FindThrift.cmake (#13337)
This is an automated email from the ASF dual-hosted git repository.
kszucs pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/master by this push:
new 5fe3a606f1 ARROW-16785: [Packaging][Linux] Add FindThrift.cmake (#13337)
5fe3a606f1 is described below
commit 5fe3a606f16b183cf706ae8d9a4adaff593c18a1
Author: Sutou Kouhei <ko...@clear-code.com>
AuthorDate: Wed Jun 8 16:51:03 2022 +0900
ARROW-16785: [Packaging][Linux] Add FindThrift.cmake (#13337)
This is a follow-up of ARROW-16721/#13292.
Authored-by: Sutou Kouhei <ko...@clear-code.com>
Signed-off-by: Krisztián Szűcs <sz...@gmail.com>
---
dev/tasks/linux-packages/apache-arrow/debian/libarrow-dev.install | 2 +-
dev/tasks/linux-packages/apache-arrow/yum/arrow.spec.in | 8 +++++++-
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/dev/tasks/linux-packages/apache-arrow/debian/libarrow-dev.install b/dev/tasks/linux-packages/apache-arrow/debian/libarrow-dev.install
index 81789b9d56..4f600692ce 100644
--- a/dev/tasks/linux-packages/apache-arrow/debian/libarrow-dev.install
+++ b/dev/tasks/linux-packages/apache-arrow/debian/libarrow-dev.install
@@ -5,7 +5,7 @@ usr/lib/*/cmake/arrow/ArrowTargets*.cmake
usr/lib/*/cmake/arrow/Find*Alt.cmake
usr/lib/*/cmake/arrow/FindArrow.cmake
usr/lib/*/cmake/arrow/FindBrotli.cmake
-usr/lib/*/cmake/arrow/Find[Suz]*.cmake
+usr/lib/*/cmake/arrow/Find[STuz]*.cmake
usr/lib/*/cmake/arrow/arrow-config.cmake
usr/lib/*/libarrow.a
usr/lib/*/libarrow.so
diff --git a/dev/tasks/linux-packages/apache-arrow/yum/arrow.spec.in b/dev/tasks/linux-packages/apache-arrow/yum/arrow.spec.in
index 8000a2b253..c671ce8aed 100644
--- a/dev/tasks/linux-packages/apache-arrow/yum/arrow.spec.in
+++ b/dev/tasks/linux-packages/apache-arrow/yum/arrow.spec.in
@@ -64,11 +64,12 @@
%define use_python (%{rhel} >= 8)
# TODO: Enable this. This works on local but is fragile on GitHub Actions and
# Travis CI.
-# %%define use_s3 (%{rhel} >= 8)
+# %%define use_s3 (%%{rhel} >= 8)
%define use_s3 0
%define have_rapidjson (%{rhel} != 8)
%define have_re2 (%{rhel} >= 8)
+%define have_thrift (%{rhel} >= 8)
%define have_utf8proc (%{rhel} >= 9)
%define have_zstd (!%{is_amazon_linux})
@@ -122,7 +123,9 @@ BuildRequires: rapidjson-devel
BuildRequires: re2-devel
%endif
BuildRequires: snappy-devel
+%if %{have_thrift}
BuildRequires: thrift-devel
+%endif
%if %{have_utf8proc}
BuildRequires: utf8proc-devel
%endif
@@ -294,6 +297,9 @@ Libraries and header files for Apache Arrow C++.
%{_libdir}/cmake/arrow/FindArrow.cmake
%{_libdir}/cmake/arrow/FindBrotli.cmake
%{_libdir}/cmake/arrow/FindSnappy.cmake
+%if %{have_thrift}
+%{_libdir}/cmake/arrow/FindThrift.cmake
+%endif
%if %{use_flight}
%{_libdir}/cmake/arrow/Findc-aresAlt.cmake
%endif