-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
Test python 3.12 #175
Test python 3.12 #175
Conversation
bec8434
to
042174b
Compare
I ran with 3.12 prior to the change in urllib3 version and it worked. The pin to that particular old urllib3 version was related to proxy http/https, as @tw4l mentioned a few days ago. This error |
This CI failure was caused by pypi timing out. I'll try it again in a while. |
Hm. I made my own branch that tests 3.12 and uses the exact urllib3 version and no bueno. |
Locally on my machine it works with python 3.12, I also made an integration branch which succeeded on github: https://github.com/white-gecko/warcio/actions/runs/10452831491/job/28944239907 |
fc1dc41
to
91bee4f
Compare
I have rebased the branch on the current master, according to my experiments that should work. |
91bee4f
to
556a6b5
Compare
The |
OK, I think this is fully safe. Thanks @white-gecko for working out this fix and also for figuring out that at some point urllib3 fixed the http/https proxy thing that motivated us pinning the version in the first place. I still have 2 questions:
|
I can answer both of my questions myself, I suppose. |
Thank you for raising the question. It might be relevant to document the decisions.
|
For future-proofing, you were installing setuptools only if the python version was exactly 3.12 ... I made it a >= while trying to get 3.13-rc to work, and that worked fine, although 3.13-rc itself failed because the greenlet wheel doesn't build. (This is typical when trying out release candidates...)
|
Requires: #172
Add python 3.12 to the build matrix.
Install
setuptools
in test environment for python 3.12 (cf. #168).