[2.x] Update virtualenv URLs to avoid blob URLs#2775
Conversation
It looks like many people are hitting the HTTP 429 for old versions of cibuildwheel. These hosted versions of virtualenv should work, let's see...
|
CI's going to be pretty broken. I can probably get copilot to figure out what parts of the 3.x CI updates to apply to the 2.x branch, seems like something it would be good at. Edit: not great, it wanted to update the deprecated cibuildwheel settings in the CI too. But I just added the parts that made sense, and went from there. |
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
15c34dc to
0fb00ee
Compare
|
@agriyakhetarpal pyodide is reporting "no such command "build"", do you know why it might be doing that? Maybe a version isn't pinned somewhere? |
|
Thanks for working on this! That failure is test_dependency_constraints_file, so I guess it's unpinning pyodide-build, causing an incompatibility. Probably that test isn't suitable in this context. btw: I'd be happy to merge this as-is, there don't appear to be any regressions and virtualenv looks like it's working! |
|
Yes, multiple versions of |
|
I did and gives us and since cibuildwheel/cibuildwheel/pyodide.py Lines 162 to 172 in faf86a6 Removing this makes the test pass, since the extensive backtracking doesn't occur. If y'all don't mind, I'll push a change here and I'll create a separate PR for the |
|
Thanks @agriyakhetarpal ! makes sense to me |
It looks like a few people are hitting the HTTP 429 for old versions of cibuildwheel. These hosted versions of virtualenv should work, let's see. If it's easy to change to this, I'm willing to do a 2.x patch for those packaging for older versions of Python.