Skip to content

Commit

Permalink
patch out precompiled headers for pyarrow
Browse files Browse the repository at this point in the history
  • Loading branch information
h-vetinari committed Jul 20, 2023
1 parent 1a627c0 commit 38be784
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 0 deletions.
3 changes: 3 additions & 0 deletions recipe/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ package:
source:
- url: https://dist.apache.org/repos/dist/release/arrow/arrow-{{ version }}/apache-arrow-{{ version }}.tar.gz
sha256: 3481c411393aa15c75e88d93cf8315faf7f43e180fe0790128d3840d417de858
patches:
# precompiled headers blow out disk space on CI agents (esp. when cross-compiling CUDA)
- patches/0001-do-not-precompile-headers-for-pyarrow.patch # [aarch64 or ppc64le]
# testing-submodule not part of release tarball
- git_url: https://github.com/apache/arrow-testing.git
git_rev: 47f7b56b25683202c1fd957668e13f2abafc0f12
Expand Down
22 changes: 22 additions & 0 deletions recipe/patches/0001-do-not-precompile-headers-for-pyarrow.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
From 14627e0663a2a8af6eb7318b11d1865ee7076460 Mon Sep 17 00:00:00 2001
From: "H. Vetinari" <[email protected]>
Date: Thu, 20 Jul 2023 22:07:54 +0200
Subject: [PATCH] do not precompile headers for pyarrow

---
python/CMakeLists.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/python/CMakeLists.txt b/python/CMakeLists.txt
index f6cc4c0e0..1505d96b4 100644
--- a/python/CMakeLists.txt
+++ b/python/CMakeLists.txt
@@ -370,7 +370,7 @@ endif()
add_library(arrow_python SHARED ${PYARROW_CPP_SRCS})
target_include_directories(arrow_python PUBLIC ${PYARROW_CPP_ROOT_DIR}
${CMAKE_CURRENT_BINARY_DIR}/pyarrow/src)
-if(NOT CMAKE_VERSION VERSION_LESS 3.16)
+if(FALSE)
target_precompile_headers(arrow_python PUBLIC
"$<$<COMPILE_LANGUAGE:CXX>:arrow/python/pch.h>")
endif()

0 comments on commit 38be784

Please sign in to comment.