-
Notifications
You must be signed in to change notification settings - Fork 197
Make the zlib-devel
package available in CentOS 8 and AmazonLinux2 images
#174
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
Conversation
The intention of this change is to improve consistency between the various platform images. The libz headers and libraries are not a dependency of Swift at runtime on any OS, but are a common need for libraries such as `swift-nio-extras`, and are provided by the Ubuntu images. The resulting images were less than 60KB larger in local testing.
@swift-ci please test |
we have had a bit of a discussion about at this point I am not convinced we should add non-core dependencies to our base image (we should look into why ubuntu has it!) given the reasons mentioned in the thread. as a quick example why that said, I would be happy to open to a wide discuss more with the community (in the forums) to explore possible solutions cc @drexin |
we discussed this further in the SSWG meeting and decided to make a one time exception for |
@swift-ci please test |
Not sure why the CI is failing... It's only failing on images I didn't touch in the first place:
Is this a known problem? |
This is was due to key server failure (connection issue) |
The intention of this change is to add to out-of-the-box usability for the new eimages and improve consistency between the various platforms. Justifications:
libz
headers and libraries are commonly required by widely-used libraries such asswift-nio-extras
and Vapor.zlib1g-dev
package is provided by all non-slim Ubuntu images.