Skip to content

Add support for PHP 8.4 for Nextcloud 33 - #2513

Closed
kowalski7cc wants to merge 5 commits into
nextcloud:masterfrom
kowalski7cc:feature/update-php-8.4
Closed

Add support for PHP 8.4 for Nextcloud 33#2513
kowalski7cc wants to merge 5 commits into
nextcloud:masterfrom
kowalski7cc:feature/update-php-8.4

Conversation

@kowalski7cc

@kowalski7cckowalski7cc commented Dec 16, 2025

Copy link
Copy Markdown

@kowalski7cc
kowalski7cc marked this pull request as draft December 16, 2025 19:36
@kowalski7cc
kowalski7ccforce-pushed the feature/update-php-8.4 branch 2 times, most recently from 75b45dc to 0b31710CompareDecember 16, 2025 19:42
@kowalski7cc
kowalski7cc marked this pull request as ready for review December 16, 2025 19:43
@kowalski7cc
kowalski7ccforce-pushed the feature/update-php-8.4 branch from 7339fbb to acba3c0CompareJanuary 2, 2026 15:52
@kowalski7cc

kowalski7cc commented Jan 2, 2026

Copy link
Copy Markdown
Author

Ok containerfile was missing libavif-dev

@joshtrichards

joshtrichards commented Jan 2, 2026

Copy link
Copy Markdown
Member

Per https://docs.nextcloud.com/server/latest/admin_manual/installation/system_requirements.html#server update PHP to 8.4

Those docs are for latest == unreleased upcoming v33

@kowalski7cc

kowalski7cc commented Jan 6, 2026

Copy link
Copy Markdown
Author

@joshtrichards I'll change the update script to reflect that. You'd prefer to set default PHP 8.4 and set NC 31 and 32 with PHP 8.3 or keep default 8.3 and set NC 33 with PHP 8.4?

@szaimenszaimen mentioned this pull request Jan 11, 2026
Open
@kowalski7cc
Add support for PHP 8.4
c4f15cc
Signed-off-by: Kowalski Dragon (kowalski7cc) <kowalski7cc@users.noreply.github.com>
@kowalski7cc
Add missing libavif
472179d
Signed-off-by: Kowalski Dragon (kowalski7cc) <kowalski7cc@users.noreply.github.com>
@kowalski7cc
kowalski7ccforce-pushed the feature/update-php-8.4 branch from acba3c0 to 301f528CompareFebruary 3, 2026 14:47
@kowalski7cc
Shift PHP 8.4 to Nextcloud 33
235b90f
Signed-off-by: Kowalski Dragon (kowalski7cc) <kowalski7cc@users.noreply.github.com>
@kowalski7cc
kowalski7ccforce-pushed the feature/update-php-8.4 branch from 301f528 to 235b90fCompareFebruary 3, 2026 14:48
@kowalski7cckowalski7cc changed the title Add support for PHP 8.4 for Nextcloud 32Add support for PHP 8.4 for Nextcloud 33Feb 3, 2026
@kowalski7cc

kowalski7cc commented Feb 3, 2026

Copy link
Copy Markdown
Author

Hello, I've changed the pull from Nextcloud 32 to NC 33 and rebased on the current master

BeneM32
BeneM32 approved these changes Feb 24, 2026
View reviewed changes
Comment thread32/fpm-alpine/Dockerfile Outdated
Comment threadDockerfile-alpine.template Outdated
Comment thread31/fpm-alpine/Dockerfile Outdated
@kowalski7cc
Merge branch 'master' into feature/update-php-8.4
ae30fc1
@kowalski7cc
Add missing new line ending
2c5a753
Signed-off-by: Kowalski Dragon (kowalski7cc) <kowalski7cc@users.noreply.github.com>
@kowalski7cc

kowalski7cc commented Feb 24, 2026

Copy link
Copy Markdown
Author

Hello @BeneM32@joshtrichards fixed the issue with newline

@BeneM32BeneM32 mentioned this pull request Feb 25, 2026
Closed
J0WI
J0WI requested changes Feb 25, 2026
View reviewed changes
Comment threadDockerfile-alpine.template
gmp \
intl \
ldap \
opcache \

@J0WIJ0WIFeb 25, 2026

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 is already enabled by default: docker-library/php#1587

Comment threadDockerfile-alpine.template
docker-php-ext-configure ftp --with-openssl-dir=/usr; \
docker-php-ext-configure gd --with-freetype --with-jpeg --with-webp; \
docker-php-ext-configure ftp %%FTP_OPTIONS%%; \
docker-php-ext-configure gd --with-freetype --with-jpeg --with-webp --with-avif; \

@J0WIJ0WIFeb 25, 2026

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.

Is AVIF used anywhere in Nextcloud with GD?

@J0WI

J0WI commented Feb 25, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

I'll fix this in #2533

@kowalski7cc

kowalski7cc commented Feb 25, 2026

Copy link
Copy Markdown
Author

ok

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@J0WIJ0WIJ0WI requested changes
+1 more reviewer
@BeneM32BeneM32BeneM32 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kowalski7cc@joshtrichards@J0WI@BeneM32