Skip to content

Minor: [C++] Default to python3 in run-test.sh - #55373

Open
filu154 wants to merge 1 commit into
apache:mainfrom
filu154:filu154/python-command-not-found
Open

filu154 wants to merge 1 commit into
apache:mainfrom
filu154:filu154/python-command-not-found

Conversation

@filu154

@filu154 filu154 commented Nov 2, 2026 •

Copy link
Copy Markdown

Rationale for this change

Allow MacOS users to run tests on a fresh install without additional setup.

Error encountered:

arrow/cpp/build-support/run-test.sh: line 101: python: command not found

What changes are included in this PR?

Change default in bash script.

Are these changes tested?

On MacOS only

Are there any user-facing changes?

No
This PR includes breaking changes to public APIs. (If there are any breaking changes to public APIs, please explain which changes are breaking. If not, you can remove this.)

This PR contains a "Critical Fix". (If the changes fix either (a) a security vulnerability, (b) a bug that caused incorrect or invalid data to be produced, or (c) a bug that causes a crash (even when the API contract is upheld), please provide explanation. If not, you can remove this.)

Was AI used for this PR?

No

In accordance to the AI generation guidelines, 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
  • AI
  • Not reviewed

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

Thanks for opening a pull request!

This pull request has been automatically converted to a draft because its title doesn't match Arrow's required format.

If this is not a minor PR, could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

After updating the title, you can mark the pull request as ready for review.

See also:

@github-actions
github-actions Bot marked this pull request as draft November 2, 2026 21:11
@filu154
filu154 marked this pull request as ready for review November 2, 2026 21:12
@github-actions
github-actions Bot marked this pull request as draft November 2, 2026 21:13
@filu154 filu154 changed the title Default to python3 [C++] Default to python3 in run-test.sh Nov 2, 2026
@filu154
filu154 force-pushed the filu154/python-command-not-found branch from 48ddfff to a57d0f2 Compare November 3, 2026 21:03
@filu154 filu154 changed the title [C++] Default to python3 in run-test.sh Minor: [C++] Default to python3 in run-test.sh Nov 3, 2026
@filu154
filu154 marked this pull request as ready for review November 3, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant