Skip to content

feat: build apisix-openresty base on alpine#97

Open
gxthrj wants to merge 30 commits into
api7:masterfrom
gxthrj:openresty/alpine
Open

feat: build apisix-openresty base on alpine#97
gxthrj wants to merge 30 commits into
api7:masterfrom
gxthrj:openresty/alpine

Conversation

@gxthrj

@gxthrj gxthrj commented Sep 22, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@gxthrj gxthrj marked this pull request as ready for review September 26, 2021 00:57

@spacewander spacewander left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please check the build result in the CI

ARG RESTY_OPENSSL_URL_BASE="https://www.openssl.org/source"
ARG RESTY_PCRE_VERSION="8.44"
ARG RESTY_J="1"
ARG RESTY_CONFIG_OPTIONS="\

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please remove unused ARG/labels

@spacewander spacewander left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you add a workflow to verify the build?

@spacewander

Copy link
Copy Markdown
Contributor

Look like we need to use glibc for Alpine?

@gxthrj

gxthrj commented Oct 15, 2021

Copy link
Copy Markdown
Contributor Author

Look like we need to use glibc for Alpine?

I used https://github.com/sgerrand/alpine-pkg-glibc , but still failed.

@@ -0,0 +1,55 @@
ARG ENABLE_PROXY=false

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This file can be removed?

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