| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| Expand Up | @@ -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", | ||
|
Comment thread
Copy link
Copy Markdown
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. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityFor 3.14 we added the classifier during the beta as well, once all tests passed. See #19199.
Sorry, something went wrong.
All reactions
Copy link
Copy Markdown
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. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityI'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
Sorry, something went wrong.
All reactions
Copy link
Copy Markdown
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. Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityLazy 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.
Sorry, something went wrong.
All reactions
|
||
| "Topic :: Software Development", | ||
| "Typing :: Typed", | ||
| ] | ||
| Expand Down | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| Expand Up | @@ -7,6 +7,7 @@ envlist = | |
| py312, | ||
| py313, | ||
| py314, | ||
| py315, | ||
| docs, | ||
| lint, | ||
| type, | ||
| Expand Down | ||
| Back | FazBrowse Home | New Git URL |
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.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualitySome librt tests only compile individual c files. If Python.h isn't included first, these tests would raise ‘_POSIX_C_SOURCE’ redefined warnings 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.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
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.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityFound a few more instances, so decided to create a separate PR after all. #21446
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.