Fix repository cache inconsistency issue#6157
Fix repository cache inconsistency issue#6157radoering wants to merge 1 commit intopython-poetry:masterfrom
Conversation
|
I fixed the same thing incidentally in #6081; the changes to the legacy repository there could be taken standalone. Our fixes are quite different: I was mostly trying to make the legacy and pypi repositories more alike and it was almost accidental that I discovered this bug. The fix here is certainly more targeted, but it does indeed make the repository classes diverge yet further. I'm pretty suspicious that this cache is more or less worthless anyway - #5868 (comment) - probably it could be removed without anyone noticing the difference... |
|
I like making There is just one thing I can't assess yet: You removed some
Same feeling, but not sure enough... |
|
I'm more than fine with you adopting those changes, help yourself. I removed the |
|
Superseded by #6165 |
|
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. |
Fixes an inconsistency when caching matches of a
LegacyRepository. Without the fixfind_packages()might return an empty list on second call for packages with only pre-releases because only (released)versionsare cached but notignored_pre_release_versions(which cannot be ignored at all events).