-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Start testing Python 3.15 #21439
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
Start testing Python 3.15 #21439
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -46,6 +46,7 @@ classifiers = [ | |
| "Programming Language :: Python :: 3.12", | ||
| "Programming Language :: Python :: 3.13", | ||
| "Programming Language :: Python :: 3.14", | ||
| "Programming Language :: Python :: 3.15", | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For 3.14 we added the classifier during the beta as well, once all tests passed. See #19199.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm okay with this, but it would be nice to make sure we support some of the headline features like lazy imports before we make a release with this classifier
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lazy imports do actually work already. At least with the "old" parser. See #20978 for more on that. I'd be ok with removing the classifier for now if that's blocking the merge for you. We can always add it later as long as we don't forget to. |
||
| "Topic :: Software Development", | ||
| "Typing :: Typed", | ||
| ] | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,6 +7,7 @@ envlist = | |
| py312, | ||
| py313, | ||
| py314, | ||
| py315, | ||
| docs, | ||
| lint, | ||
| type, | ||
|
|
||
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.
Some librt tests only compile individual c files. If
Python.hisn't included first, these tests would raise‘_POSIX_C_SOURCE’ redefinedwarnings on Python 3.15.See https://bugzilla.redhat.com/show_bug.cgi?id=2416110 for a similar issue
https://docs.python.org/3/c-api/intro.html#include-files
--
No need to backport anything specifically as building librt completely works fine already. It's just if only individual files are compiled.
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.
Found a few more instances, so decided to create a separate PR after all. #21446