8000 Update installation.md Python 3 command to `python3` by daj · Pull Request #5713 · aframevr/aframe · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update installation.md Python 3 command to python3 #5713

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 2 commits into from
May 7, 2025

Conversation

daj
Copy link
Contributor
@daj daj commented May 5, 2025

Description:

When following the instructions on macOS 15.4, after installing the recommended version of Python from python.org, I found that the Python 3 executable is python3, not python.

Some quick internet searching suggests that python3 is probably the most reliable command to run across multiple OSes.

I confirmed the command seems to work as expected:

% python3 -m http.server
Serving HTTP on :: port 8000 (http://[::]:8000/) ...
::1 - - [04/May/2025 23:10:46] "GET /my-test-vr.html HTTP/1.1" 200 -

Changes proposed:

Simple docs change from python to python3.

When following the instructions on macOS, after installing the recommended version of Python from python.org, I found that the Python 3 executable is `python3`, not `python`.

Some quick internet searching suggests that `python3` is probably the most reliable command to run across multiple OSes.
@dmarcos
Copy link
Member
dmarcos commented May 5, 2025

@mrxz @vincentfretin Any thoughts?

@vincentfretin
Copy link
Contributor

It's the same on latest ubuntu, at least on 22.04, there is no python alias anymore (that was pointing to python2), you need to execute python3 command or python2 explicitly.
I personally would get rid of the python -m SimpleHTTPServer completely in the doc now and only document the python3 -m http.server command. Python 2 end of life was January 1st, 2020.

@mrxz
Copy link
Contributor
mrxz commented May 6, 2025

As far as I've encountered, it's common to have python3 and python2 for their respective versions, with python symlinking to the "default" one, though virtually always python 2 for backwards compatibility.

I agree with @vincentfretin that we should just list the python3 command and remove the old one.

@dmarcos
Copy link
Member
dmarcos commented May 6, 2025

@daj can we switch to python3 as described above?

@daj
Copy link
Contributor Author
daj commented May 7, 2025

@dmarcos I updated it, please let me know if I need to make any other changes.

@dmarcos
Copy link
Member
dmarcos commented May 7, 2025

Thanks @dan and congrats on your first contribution 👏

@dmarcos dmarcos merged commit f47d94d into aframevr:master May 7, 2025
2 checks passed
@daj daj deleted the patch-1 branch May 8, 2025 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0