Skip to content

Conversation

@radarhere
Copy link
Member

No description provided.

IMAGENAME := $(if $(DOCKER_USERNAME), $(DOCKER_USERNAME)/$(TARGET), $(TARGET))
BRANCH := $(shell git rev-parse --abbrev-ref HEAD)
TEST_TARGET = ubuntu-24.04-noble-amd64
TEST_TARGET = fedora-41-amd64
Copy link
Member

Choose a reason for hiding this comment

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

Because Fedora 41 comes with Python 3.13, and Ubuntu 24.04 comes with 3.12?

I've not been using these manylinux images locally, what's the other pros/cons of switching?

Does it affect the main Pillow CI?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, the Python version is why.

I don't expect there to be any side effects of changing the test image - it won't affect the build itself. As the README says

The test target here is mainly to validate the image build, it is assumed that the builds will be used for other purposes or tested in other images.

The wheel build jobs aren't used by the main repository.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks!

@radarhere radarhere changed the title Added Python 3.13 to manylinux images Added Python 3.13 to manylinux build images Nov 9, 2024
@hugovk hugovk merged commit 4c74c2a into python-pillow:main Nov 9, 2024
@radarhere radarhere deleted the python313 branch November 10, 2024 10:31
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