turn on --installp-pip test by youknowone · Pull Request #3795 · RustPython/RustPython
RustPython / RustPython Public
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Merged
Merged
Conversation
youknowone
commented
Jun 16, 2022
youknowone commented
Member Author
hmm, what's left?
fanninpm
commented
Jun 17, 2022
fanninpm commented
Contributor
Perhaps a rebase?
youknowone
force-pushed
the
install-pip
branch
from
f2785d4 to
43b3f8e
Compare
youknowone
commented
Jun 17, 2022
youknowone commented
Member Author
rebase is not helpful
dieterplex
commented
Jun 22, 2022
dieterplex commented
Contributor
Dig into get-pip.py and found that html parser not works. no links could be parsed from the page https://pypi.org/simple/pip/
I guess we should check html.parser.HTMLParser first.
Test with
> target/release/rustpython get-pip-zip-extracted/__main__.py -vvv install --upgrade --force-reinstall -t site-packages/ pip
specific source at line 353: parser.feed(page.content.decode(encoding)) from get-pip-zip-extracted/_internal/index/collector.py:parse_links().
There is no item in parser.anchor after feed.
youknowone
commented
Jun 22, 2022
youknowone commented
Member Author
Thank you!
fanninpm
mentioned this pull request
youknowone
force-pushed
the
install-pip
branch
from
43b3f8e to
9490603
Compare
youknowone
commented
Jul 24, 2022
youknowone commented
Member Author
try again
youknowone
merged commit
8ae1c70
into
RustPython:main
youknowone
deleted the
install-pip
branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment