[Pillow] Removed zlib1g-dev installation#5950
Conversation
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
|
@googlebot I signed it! |
DavidKorczynski
left a comment
There was a problem hiding this comment.
LGTM. The issues with the CI is unrelated to Pillow. I have checked the build passes locally.
|
@jonathanmetzman this one is good to get merged in - the CI currently fails due to the issue where additional projects gets build in the CI (in this case freeradius) which fail. |
Pillow recently started building zlib on Linux in it's wheels, and that is conflicting with the installation of zlib1g-dev from our oss-fuzz Dockerfile.
Copying files from here as a test, it fails with the same error, and then with this PR, it ... well, it doesn't pass exactly, but it makes it to "FUZZING_LANGUAGE: unbound variable", which I presume is just an artefact of my testing setup.
So this PR removes zlib1g-dev from the list of apt-get installations.
cc @wiredfool