-
-
Notifications
You must be signed in to change notification settings - Fork 153
Update dependencies to support python 3.13 #249
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
base: master
Are you sure you want to change the base?
Conversation
This change updates all first-order dependencies to newer versions which support newer versions of python. Fix formatting while here to use 4-space indents instead of hard tabs, per PEP8. Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>
This change removes 3.8 pins as that version of python is no longer supported by the upstream python project. Remove outdated OS X instructions while here and do some minor formatting tweaks with h1 and h2 sections. Remove the manually provided ToC while here: GitHub automatically generates the ToC for "free". Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>
... until the update is pushed upstream in ArchiveTeam/grab-site#249. Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>
... is bumped as part of ArchiveTeam/ludios_wpull#36 . Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>
8b7a66a to
e81eed8
Compare
| "google-re2>=1.0.6", | ||
| "websockets>=6.0", | ||
| "click>=8.0.0", | ||
| "wpull @ https://github.com/ngie-eign/ludios_wpull/archive/py313-support.zip", |
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.
Would you consider switching that to e.g.
| "wpull @ https://github.com/ngie-eign/ludios_wpull/archive/py313-support.zip", | |
| "wpull~=5.0.4", |
?
The MR ArchiveTeam/ludios_wpull#36 is merged + released now.
And it seems that this
"wpull @ https://github.com/ngie-eign/ludios_wpull/archive/py313-support.zip",doesn't work anymore
$ uvx --python=3.8 --no-binary-package lxml git+https://github.com/ngie-eign/grab-site@py313-support
Updated https://github.com/ngie-eign/grab-site (e81eed8c37616492466848ab5ed68104c7339693)
× Failed to download and build `wpull @ https://github.com/ngie-eign/ludios_wpull/archive/py313-support.zip`
├─▶ Failed to fetch: `https://github.com/ngie-eign/ludios_wpull/archive/py313-support.zip`
╰─▶ HTTP status client error (404 Not Found) for url (https://codeload.github.com/ngie-eign/ludios_wpull/zip/py313-support)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.
Actually, wpull~=5.0.4 does not exist in PyPI (https://pypi.org/project/wpull/ only a very old version of it).
I guess the correct solution would be
git+https://github.com/ArchiveTeam/ludios_wpull@5.0.4
This change updates all pins to work with software that supports python 3.13, in an effort to add support for new supported versions of python.
This change doesn't go in-depth to burn any bridges associated with unsupported versions of python, e.g., pre-3.9. This should be handled in a separate change.
The proposed change in the PR is potentially rough around the edges and followup commits may need to be performed to make the code function fully with python 3.8+, however the likelihood of things functioning out of the box with 3.9/3.10 is high.
Depends on: ArchiveTeam/ludios_wpull#36.
Closes: #245.