Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma… #1598

Merged
merged 1 commit into from Oct 8, 2018
Merged

Dockerfiles for Alpine 3.8/edge build with Nginx 1.14/1.15 (stable/ma… #1598

merged 1 commit into from Oct 8, 2018

Conversation

nberlee
Copy link
Contributor

@nberlee nberlee commented Sep 24, 2018

…inline), (directory structure organised for DockerHub build compatibility).

Alpine Linux is a security-oriented, lightweight Linux distribution based on musl libc and busy box. Alpine is therefor the most logical OS base for Docker Images.

Although it does not fix #1181 completely, it helps to overcome the problems of building the PSOL on Alpine,

…inline), (directory structure organised for DockerHub build compatibility).

Alpine Linux is a security-oriented, lightweight Linux distribution based on musl libc and busy box. Alpine is therefor the most logical OS base for Docker Images.

Although it does not fix #1181 completely, it helps to overcome the problems of building the PSOL on Alpine,
Copy link
Contributor

@jmarantz jmarantz left a comment

Choose a reason for hiding this comment

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

Should we also go ahead & upgrade the trunk to png 1.6 to avoid having to do this patch?

@oschaaf
Copy link
Member

oschaaf commented Oct 3, 2018

Re: "Should we also go ahead & upgrade the trunk to png 1.6 to avoid having to do this patch?"
That would be nice, I'll have a go at that.

@oschaaf
Copy link
Member

oschaaf commented Oct 3, 2018

(actually, mps already has libpng 1.6 on the bleeding edge version: https://github.com/apache/incubator-pagespeed-mod/tree/master/third_party/libpng)

@oschaaf
Copy link
Member

oschaaf commented Oct 4, 2018

Maybe we should also link to the readme in https://github.com/apache/incubator-pagespeed-ngx/blob/master/README.md, and the docs (a new page over at https://www.modpagespeed.com/doc)?

@oschaaf oschaaf merged commit c24e387 into apache:master Oct 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Alpine
3 participants