GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-100972: For command line examples in the documentation, use "python" rather than "python3" by furkanonder · Pull Request #100973 · python/cpython

Conversation

@furkanonder

@furkanonder furkanonder commented Jan 12, 2023 •
edited by bedevere-bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor
  • Issue: For command line examples in the documentation, use "python" rather than "python3" #100972

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@furkanonder
Prefer "python" over "python3"
bd6b3ff
@furkanonder
furkanonder requested a review from vsajip as a code owner January 12, 2023 09:49
@bedevere-bot bedevere-bot added the awaiting review label Jan 12, 2023
@bedevere-bot bedevere-bot mentioned this pull request Jan 12, 2023
For command line examples in the documentation, use "python" rather than "python3" #100972
Closed
@bedevere-bot bedevere-bot added docs Documentation in the Doc dir skip news labels Jan 12, 2023
@furkanonder furkanonder changed the title gh-100972: Prefer "python" over "python3" gh-100972: For command line examples in the documentation, use "python" rather than "python3 Jan 12, 2023
@hugovk hugovk mentioned this pull request Jan 12, 2023
Prefer "python" over "python3" for command line examples in docs. #98763
Closed
hugovk
hugovk approved these changes Jan 12, 2023
View reviewed changes

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@bedevere-bot bedevere-bot added awaiting merge and removed awaiting review labels Jan 12, 2023
felixxm
felixxm reviewed Jan 12, 2023
View reviewed changes
Comment thread Doc/library/importlib.metadata.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
felixxm
felixxm reviewed Jan 12, 2023
View reviewed changes
Comment thread Doc/library/__main__.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
felixxm
felixxm reviewed Jan 12, 2023
View reviewed changes
Comment thread Doc/tutorial/venv.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
felixxm
felixxm reviewed Jan 12, 2023
View reviewed changes
Comment thread Doc/whatsnew/3.9.rst

* Python now gets the absolute path of the script filename specified on
the command line (ex: ``python3 script.py``): the ``__file__`` attribute of
the command line (ex: ``python script.py``): the ``__file__`` attribute of

@felixxm felixxm Jan 12, 2023

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see much value in changing old release notes.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
hauntsaninja and erlend-aasland reacted with thumbs up emoji
All reactions
  • 👍 2 reactions
@vsajip

vsajip commented Jan 12, 2023

Copy link
Copy Markdown
Member

Doesn't this assume that the system Python installation will be Python 3.x such that "python" will start a 3.x interpreter? If that's the case, it shouldn't be needed to specify "python3" for "-m venv" usages, right? Also, is it definitely the case that "python" will invoke 3.x on all platforms, old and new? For example I have some (not very) old Ubuntu machines where "python3" is needed. Anyone following along the documentation with this change could run into problems.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@rhettinger

rhettinger commented Jan 14, 2023

Copy link
Copy Markdown
Contributor

On my Mac, python no longer refers to an executable. Only python3 works. So, at least for Mac Users, the status quo is better. With the proposed substitutions, the examples won't work any more.
MonadChains reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@furkanonder

furkanonder commented Jan 15, 2023

Copy link
Copy Markdown
Contributor Author

On my Mac, python no longer refers to an executable. Only python3 works. So, at least for Mac Users, the status quo is better. With the proposed substitutions, the examples won't work any more.

If there are problems with the Mac, I think this PR should be reverted.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@AlexWaygood
Merge branch 'main' into issue-100972
2c3003a
@hugovk hugovk changed the title gh-100972: For command line examples in the documentation, use "python" rather than "python3 gh-100972: For command line examples in the documentation, use "python" rather than "python3" Apr 7, 2023
@erlend-aasland

erlend-aasland commented Jun 19, 2023

Copy link
Copy Markdown
Contributor

On my Mac, python no longer refers to an executable. Only python3 works. So, at least for Mac Users, the status quo is better. With the proposed substitutions, the examples won't work any more.

If there are problems with the Mac, I think this PR should be reverted.

I disagree; reverting that PR will only create more churn (both in the git history, but more importantly also by burning CI and contributor time). I'm closing this and the linked issue:

  • There is no uncontroversial way to fix this
  • Many core devs will categorise spelling fixes (including this) as churn; even more so given there is no consensus
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@erlend-aasland erlend-aasland closed this Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hugovk hugovk hugovk approved these changes
@vsajip vsajip Awaiting requested review from vsajip
+2 more reviewers
@felixxm felixxm felixxm left review comments
@MonadChains MonadChains MonadChains left review comments
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

awaiting merge docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

9 participants

@furkanonder @vsajip @rhettinger @erlend-aasland @hugovk @felixxm @MonadChains @bedevere-bot @AlexWaygood
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.