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

chore: upgrade dockerfile #144

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

soulteary
Copy link
Contributor

issue #143

  • support php8
  • Complete Nginx image

hope it helps @rryqszq4

COPY third_party /build/third_party

# Nginx Docker Playground (https://github.com/nginx-with-docker/nginx-docker-playground)
FROM soulteary/prebuilt-nginx-modules:base-1.23.1-alpine AS Builder
Copy link
Contributor

Choose a reason for hiding this comment

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

Please, try to use official docker images.

Copy link
Owner

Choose a reason for hiding this comment

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

@soulteary I think joanhey is right. haha

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree with this change, and will update this PR once I've finished a piece of content with a better scene, hopefully within the last two months of this year.

eg: https://www.zhihu.com/question/21483073/answer/2647942304

@rryqszq4 @joanhey


# Nginx Docker Playground (https://github.com/nginx-with-docker/nginx-docker-playground)
FROM soulteary/prebuilt-nginx-modules:base-1.23.1-alpine AS Builder
RUN apk update && apk --no-cache add curl gcc g++ make musl-dev linux-headers gd-dev geoip-dev libxml2-dev libxslt-dev openssl-dev pcre-dev perl-dev pkgconf zlib-dev libedit-dev ncurses-dev php8-dev php8-embed git unzip argon2-dev
Copy link
Contributor

Choose a reason for hiding this comment

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

Add OPcache extension

@rryqszq4
Copy link
Owner

@soulteary Maybe github-actions-build was wrong, wait for repair.

@rryqszq4
Copy link
Owner

rryqszq4 commented Apr 14, 2023

Best @soulteary
Github-actions-build has been repaired, can you try again and touch checks.
And by @joanhey github-actions-build changes and fastly, at here https://github.com/rryqszq4/ngx-php/blob/master/.github/workflows/ngx_php.yml

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.

3 participants