Skip to content

pre-push.sh: Use python3 if python is not found #95722

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 12, 2022

Conversation

xu-cheng
Copy link
Contributor

@xu-cheng xu-cheng commented Apr 6, 2022

Since Python 2 has reached EOL, python may not be available in certain systems (e.g., recent macOS). We should use python3 in this case to avoid error like python: No such file or directory.

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 6, 2022
@Mark-Simulacrum
Copy link
Member

r=me with commits squashed

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 11, 2022
Since Python 2 has reached EOL, `python` may not be available in certain
systems (e.g., recent macOS). We should use `python3` in this case to
avoid error like `python: No such file or directory`.
@xu-cheng
Copy link
Contributor Author

Commits are squashed now.

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 11, 2022

📌 Commit 365d4f4 has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 11, 2022
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Apr 12, 2022
pre-push.sh: Use python3 if python is not found

Since Python 2 has reached EOL, `python` may not be available in certain systems (e.g., recent macOS). We should use `python3` in this case to avoid error like `python: No such file or directory`.
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 12, 2022
…askrgr

Rollup of 7 pull requests

Successful merges:

 - rust-lang#95320 (Document the current MIR semantics that are clear from existing code)
 - rust-lang#95722 (pre-push.sh: Use python3 if python is not found)
 - rust-lang#95881 (Use `to_string` instead of `format!`)
 - rust-lang#95909 (rustdoc: Reduce allocations in a `theme` function)
 - rust-lang#95910 (Fix crate_type attribute to not warn on duplicates)
 - rust-lang#95920 (use `Span::find_ancestor_inside` to get right span in CastCheck)
 - rust-lang#95936 (Fix a bad error message for `relative paths are not supported in visibilities` error)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit a899ebd into rust-lang:master Apr 12, 2022
@rustbot rustbot added this to the 1.62.0 milestone Apr 12, 2022
@xu-cheng xu-cheng deleted the pre-push branch April 12, 2022 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants