-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
remove SimpleRepositoryPage #9166
remove SimpleRepositoryPage #9166
Conversation
For completeness, the trailing slash comes from PEP 503. The intention with the separation was to allow non compliant repos like this one. Seems somewhere along the lines that support went away causing the single page repos to fail after 1.2.
The html page was indeed originally intended to abstract away both cases. |
See #5517 |
https://storage.googleapis.com/jax-releases/jax_releases.html has absolute links to wheels, so the bogus trailing slash made no difference in the example that #5517 tested against. single-page repositories with relative links have never worked. |
#5517 makes use of Fwiw, I am not saying |
For this one, I am fairly certain this was added to avoid redirects, which I think we handle much better these days implicitly. |
the
It really is the handling of the |
f51ff21
to
11d09e7
Compare
I added a testcase to show that this fixes something (and protect against regressions) |
f0f31b5
to
8e2bf84
Compare
8e2bf84
to
2df6ba7
Compare
1c8ab4a
to
0836e48
Compare
0836e48
to
19d8d06
Compare
19d8d06
to
14ca326
Compare
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
so far as I can see the only achievement of the
SimpleRepositoryPage
is to add an unwanted trailing slash, making it impossible to use files rather than directories as single page sources eghttps://download.pytorch.org/whl/torch_stable.html
the addition of a trailing slash was made back in pre-history at 1900633. I can't tell what the intention was: but for what it's worth the two testcases in that commit survive today and still pass.
fixes #6885