8000 Use host CPython as Python dependency; remove xcb image by indygreg · Pull Request #627 · astral-sh/python-build-standalone · GitHub
[go: up one dir, main page]

Skip to content

Use host CPython as Python dependency; remove xcb image #627

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

Merged
merged 2 commits into from
May 26, 2025

Conversation

indygreg
Copy link
Collaborator
@indygreg indygreg commented May 26, 2025

The xcb Dockerfile / image existed to provide a host build of CPython, which was required to build libxcb, xcb-proto, and tk. The existence of this image predated our building a host CPython in order to build CPython itself.

With PBS now unconditionally building a host CPython, we can use our host CPython during the builds of these packages, eliminating the need for the xcb image.

This commit makes that transition.

We also upgrade libxcb and xcb-proto to the latest version as part of this migration. Older versions of libxcb aren't compatible with Python 3.12+ due to removal of the imp module.

We stopped shipping the distributions in 6fa8fe9
in October 2023. At the time we preserved build support just in case.

I don't believe anyone has complained about the loss of x86 Linux distributions.

I don't believe there is compelling value in PBS supporting x86 Linux
distributions.

If someone wants to run Python on x86 Linux, they can always use CPython's
official build systen, which still supports targeting x86 Linux.

This commit rips out our support building for x86 Linux.
@indygreg indygreg force-pushed the indygreg/remove-xcb-image branch 2 times, most recently from 8523d50 to 3d1fec5 Compare May 26, 2025 20:21
The `xcb` Dockerfile / image existed to provide a host build of CPython,
which was required to build libxcb, xcb-proto, and tk. The existence of this
image predated our building a host CPython in order to build CPython itself.

With PBS now unconditionally building a host CPython, we can use our host
CPython during the builds of these packages, eliminating the need for the
xcb image.

This commit makes that transition.

We also upgrade libxcb and xcb-proto to the latest version as part of
this migration. Older versions of libxcb aren't compatible with Python
3.12+ due to removal of the `imp` module.
@indygreg indygreg force-pushed the indygreg/remove-xcb-image branch from 3d1fec5 to 79cd8bd Compare May 26, 2025 20:32
@indygreg indygreg marked this pull request as ready for review May 26, 2025 21:09
Copy link
Collaborator
@geofft geofft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, the xcb image confused me :)

Base automatically changed from indygreg/remove-linux-i686 to main May 26, 2025 23:24
@indygreg indygreg merged commit 35eec28 into main May 26, 2025
435 checks passed
@indygreg indygreg deleted the indygreg/remove-xcb-image branch May 26, 2025 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0