Remove xz support entirely#19143
Conversation
woodruffw
force-pushed
the
ww/rm-bz-lzma-in-zips
branch
from
fec0ee5 to
a38cfe0
Compare
| TarGz, | ||
| Legacy(LegacySourceDistExtension), |
There was a problem hiding this comment.
Flagging: this is technically not necessary, but I've done this to provide some more type-level clarity around how sdists are parsed -- we still parse and accept these non-PEP 625 distribution filenames, but we now explicitly bucket them into the Legacy enum to simplify the "this is a legacy behavior" distinction elsewhere.
| // TODO: Remove these other extensions in the future. | ||
| // NOTE: These get parsed from network sources like PyPI, so we don't | ||
| // necessarily want to hard-fail, just skip in the future. |
There was a problem hiding this comment.
Flagging: this is done in prioritized_distribution.rs in this PR now 🙂
|
Noting: one longer term thing we might want to do here is encode more of this at the type level, i.e. have a |
Signed-off-by: William Woodruff <william@astral.sh>
2 participants
Summary
This removes our support for xz decompression entirely, including in the tarball pathway.
WIP.Atop #18927.Test Plan
Adjust the current tests for new failures.