Skip to content

Drop dev-python/pillow from SDK#3759

Merged
krnowak merged 4 commits intomainfrom
krnowak/drop-pillow
Feb 27, 2026
Merged

Drop dev-python/pillow from SDK#3759
krnowak merged 4 commits intomainfrom
krnowak/drop-pillow

Conversation

@krnowak
Copy link
Member

@krnowak krnowak commented Feb 27, 2026

In next weekly updates batch, pillow gained a dependency on
dev-python/pybind11, which pulls in dev-cpp/eigen, which pulls in some
eclasses and more deps. This is too annoying for a tool that is
effectively of no use for Flatcar. We should be able to get away with
just "providing" the package, since docutils (the only package that
pulls dev-python/pillow) is documented to handle missing pillow
gracefully.

CI: http://localhost:8080/job/container/job/sdk/2523

In next weekly updates batch, pillow gained a dependency on
dev-python/pybind11, which pulls in dev-cpp/eigen, which pulls in some
eclasses and more deps. This is too annoying for a tool that is
effectively of no use for Flatcar. We should be able to get away with
just "providing" the package, since docutils (the only package that
pulls dev-python/pillow) is documented to handle missing pillow
gracefully.

Signed-off-by: Krzesimir Nowak <knowak@microsoft.com>
Signed-off-by: Krzesimir Nowak <knowak@microsoft.com>
It was a dependency of the dev-python/pillow package, which we
dropped.

Signed-off-by: Krzesimir Nowak <knowak@microsoft.com>
Signed-off-by: Krzesimir Nowak <knowak@microsoft.com>
@krnowak krnowak requested a review from a team as a code owner February 27, 2026 12:11
Copy link
Contributor

@tormath1 tormath1 left a comment

Choose a reason for hiding this comment

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

What about dropping docutils as well?

@krnowak
Copy link
Member Author

krnowak commented Feb 27, 2026

What about dropping docutils as well?

I can try that too, but rather in a separate PR. More packages depend on docutils, so there is a lower chance that all of them will handle missing docutils gracefully.

@krnowak
Copy link
Member Author

krnowak commented Feb 27, 2026

CI is past the SDK and packages build, which means that missing pillow was handled gracefully by docutils.

@krnowak krnowak merged commit 3eb73a0 into main Feb 27, 2026
1 of 5 checks passed
@krnowak krnowak deleted the krnowak/drop-pillow branch February 27, 2026 20:55
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