Skip to content
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

Can we use httpd on the base stack? #322

Closed
ryanmoran opened this issue May 2, 2022 · 5 comments
Closed

Can we use httpd on the base stack? #322

ryanmoran opened this issue May 2, 2022 · 5 comments
Assignees

Comments

@ryanmoran
Copy link
Member

Summary

The HTTPD dependency that this buildpack installs appears to be unable to run on the base stack. Can we complete an exploration to determine why we cannot run this on base and if it might be reasonable for us to get this to run on base?

@TisVictress
Copy link
Contributor

TisVictress commented Jun 15, 2022

@ryanmoran Adding libexpat1 to mixins for the base stack seems to work. I didn't see that any other package was required to get the httpd integration test to work (unless I'm missing something). I created this draft pull-request for the stacks repo. And here's a draft for the base-builder.

If we want to add support for the php-httpd use case, we'll need to add the following: libargon2-0, libcurl4, libedit2, libgd3, libmagickwand-6.q16-3, libonig4, libxml2

Should I update the stacks PR to include these as well?

@ryanmoran
Copy link
Member Author

No need for the extra dependencies to support php-httpd since the PHP buildpack isn't supported on Base.

@TisVictress
Copy link
Contributor

libexpat1 is added to both build and run for symmetry.
The difference between the sizes of the build and run images after including libexpat1 is negligible:

w/ libexpat1 w/o libexpat1
Run 88.996 MB 88.595 MB
Build 326.075 MB 326.101 MB

@TisVictress
Copy link
Contributor

The PR for the base builder: https://github.com/paketo-buildpacks/base-builder/pull/485/files

@TisVictress
Copy link
Contributor

Closing. This was resolved when the PR to base-builder was merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants