Skip to content

Conversation

@x1sc0
Copy link

@x1sc0 x1sc0 commented Oct 31, 2025

They were removed upstream in
tukaani-project/xz@85b081f

@x1sc0 x1sc0 requested a review from a team as a code owner October 31, 2025 10:27
@bedevere-app
Copy link

bedevere-app bot commented Oct 31, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@x1sc0 x1sc0 changed the title lzma: crc64_table.c and crc32_table.c lzma: do not include crc64_table.c and crc32_table.c Oct 31, 2025
@picnixz
Copy link
Member

picnixz commented Oct 31, 2025

The change is incorrect. This needs to be synced with https://github.com/python/cpython-source-deps and https://github.com/python/cpython-source-deps/tree/xz, but we pinned the version for now. In addition, we'll need to create an issue. Closing.

@picnixz picnixz closed this Oct 31, 2025
@picnixz
Copy link
Member

picnixz commented Oct 31, 2025

@emmatyping Do we need to bump xz dependency?

@emmatyping
Copy link
Member

@picnixz looks like yes
image

(source-deps currently has 5.2.5)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants