-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
build: support python 3.10.0 #40296
build: support python 3.10.0 #40296
Conversation
typo: |
Thank you. |
Also in commit message. |
5cd792d
to
4007ded
Compare
Thank you. |
Please update the references in |
Thank you. |
BUILDING.md
Outdated
@@ -235,7 +235,7 @@ The Node.js project supports Python >= 3 for building and testing. | |||
|
|||
* `gcc` and `g++` >= 8.3 or newer, or | |||
* GNU Make 3.81 or newer | |||
* Python 3.6, 3.7, 3.8, or 3.9 (see note above) | |||
* Python 3.6, 3.7, 3.8, 3.9 or 3.10 (see note above) |
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.
* Python 3.6, 3.7, 3.8, 3.9 or 3.10 (see note above) | |
* Python 3.6, 3.7, 3.8, 3.9, or 3.10 (see note above) |
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.
Since this list is getting pretty long maybe we could just specify a range instead of listing each individual version?
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 think 5 versions of the list listing are still acceptable.
BUILDING.md
Outdated
@@ -250,7 +250,7 @@ FreeBSD and OpenBSD users may also need to install `libexecinfo`. | |||
#### macOS prerequisites | |||
|
|||
* Xcode Command Line Tools >= 11 for macOS | |||
* Python 3.6, 3.7, 3.8, or 3.9 (see note above) | |||
* Python 3.6, 3.7, 3.8, 3.9 or 3.10 (see note above) |
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.
* Python 3.6, 3.7, 3.8, 3.9 or 3.10 (see note above) | |
* Python 3.6, 3.7, 3.8, 3.9, or 3.10 (see note above) |
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.
Thank you for your suggestions.
@nodejs/python |
ee8c753
to
c3d5917
Compare
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.
Let’s hold on this until Python 2.10 is released https://www.python.org/downloads which is scheduled to happen today. It should be tested by at least one Linux, macOS, and Windows user and in our GitHub Actions.
c3d5917
to
3bff9b5
Compare
Thank you for your key suggestion. I will add all tests once the 3.10.0 has been released. |
Python 3.10.0 is now available https://pythoninsider.blogspot.com/2021/10/python-3100-is-available.html |
Yeah. It. seems like the list of GitHub is still not available. |
@@ -11,7 +11,7 @@ jobs: | |||
max-parallel: 8 | |||
matrix: | |||
os: [macos-latest, ubuntu-latest] # , windows-latest] | |||
python-version: [3.6, 3.7, 3.8, 3.9] | |||
python-version: [3.6, 3.7, 3.8, 3.9, 3.10] |
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.
tools/gyp changes should be done in https://github.com/nodejs/gyp-next
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.
Testing with nodejs/gyp-next#127 ... The first commit demonstrates that if 3.10 is not quoted (i.e. "3.10") in the yaml then actions/setup-python
will attempt (and fail) to load Python 3.1 instead of 3.10.
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.
tools/gyp changes should be done in https://github.com/nodejs/gyp-next
Thank you.
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: nodejs#40296 Fixes: nodejs#40294 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
Allow python 3.10.0
Fixes: #40294