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

chaokunyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-fury.git


The following commit(s) were added to refs/heads/main by this push:
     new c0a28d1a [C++] Use python3 instead of python in bazel build (#1297)
c0a28d1a is described below

commit c0a28d1aa8d68772bbf5427c369442d1ea39712b
Author: Twice <[email protected]>
AuthorDate: Wed Jan 3 23:07:13 2024 +0800

    [C++] Use python3 instead of python in bazel build (#1297)
    
    Some linux distributions do not have `python` by default, and some make
    `python` point to python2 instead of python3.
---
 bazel/arrow/pyarrow_configure.bzl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/bazel/arrow/pyarrow_configure.bzl 
b/bazel/arrow/pyarrow_configure.bzl
index 5ff96b3e..8875ba15 100644
--- a/bazel/arrow/pyarrow_configure.bzl
+++ b/bazel/arrow/pyarrow_configure.bzl
@@ -171,7 +171,7 @@ def _symlink_genrule_for_dir(
     )
     return genrule
 
-def _get_pyarrow_include(repository_ctx, python_bin="python"):
+def _get_pyarrow_include(repository_ctx, python_bin="python3"):
     """Gets the pyarrow include path."""
     result = _execute(
         repository_ctx, [
@@ -183,7 +183,7 @@ def _get_pyarrow_include(repository_ctx, 
python_bin="python"):
             + python_bin + ".) " + "Is distutils installed?"))
     return result.stdout.splitlines()[0]
 
-def _get_pyarrow_shared_library(repository_ctx, library_name, 
python_bin="python"):
+def _get_pyarrow_shared_library(repository_ctx, library_name, 
python_bin="python3"):
     """Gets the pyarrow shared library path."""
     code = """import pyarrow, os, glob;print(glob.glob(os.path.join(""" +\
         """os.path.dirname(pyarrow.__file__), 
'lib{}.*'))[0])""".format(library_name)
@@ -198,7 +198,7 @@ def _get_pyarrow_shared_library(repository_ctx, 
library_name, python_bin="python
     return result.stdout.splitlines()[0]
 
 #python numpy include
-def _get_python_numpy_include(repository_ctx, python_bin="python"):
+def _get_python_numpy_include(repository_ctx, python_bin="python3"):
     """Gets the python numpy include path."""
     result = _execute(
         repository_ctx, [


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to