Skip to content

PHP 8.5 : Remove install Opcache (it's now native) #454

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.

Sign up for GitHub

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

Jump to bottom
Merged
jolelievre merged 1 commit into from
Nov 21, 2025
Merged

PHP 8.5 : Remove install Opcache (it's now native) #454

jolelievre merged 1 commit into from
Nov 21, 2025

Conversation

@Progi1984
Copy link
Member

@Progi1984 Progi1984 commented Nov 21, 2025

Questions Answers
Branch? master
Description? PHP 8.5 : Remove install Opcache (it's now native)
Type? bug fix
Category? CO
BC breaks? no
Deprecations? no
How to test? CI is 🟢
Fixed issue or discussion? N/A
Related PRs N/A
Sponsor company lefevre.dev

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Nov 21, 2025
jolelievre
jolelievre approved these changes Nov 21, 2025
View reviewed changes
Copy link
Contributor

@jolelievre jolelievre left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Ok for the quick modification

But these files are generated by the ./generate_tags.sh script usually, the script will need an improvement to handle the PHP 8.5 special use case and remove this extra line for the base image

@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Nov 21, 2025
@jolelievre jolelievre merged commit ae19ab5 into master Nov 21, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Nov 21, 2025
@jolelievre jolelievre deleted the php85 branch November 21, 2025 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jolelievre jolelievre jolelievre approved these changes

Assignees

No one assigned

Labels

Waiting for QA

Projects

PR Dashboard
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Progi1984 @jolelievre @ps-jarvis