Skip to content

Conversation

@JanTvrdik
Copy link

Summary

  • Add PHP 8.5 support following the same pattern as PHP 8.4 (commit c1c00dc)
  • Add extensions/8.5 directory with symlinks to core extensions
  • Update CI workflow to build and test PHP 8.5 images
  • Update Makefile with PHP 8.5 test targets

Test plan

  • Verify PHP 8.5 images build successfully
  • Run test suite against PHP 8.5 images
  • Verify extensions work correctly on PHP 8.5

@JanTvrdik JanTvrdik marked this pull request as ready for review November 26, 2025 11:32
if [[ "${PHP_VERSION}" == "8.4" ]]; then
echo "-- PHP8.4 not support yet blackfire : https://support.blackfire.platform.sh/hc/en-us/articles/4740726712082-PHP-versions-Platforms-support"
if [[ "${PHP_VERSION}" == "8.4" ]] || [[ "${PHP_VERSION}" == "8.5" ]]; then
echo "-- PHP${PHP_VERSION} not support yet blackfire : https://support.blackfire.platform.sh/hc/en-us/articles/4740726712082-PHP-versions-Platforms-support"
Copy link
Collaborator

Choose a reason for hiding this comment

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

issue: Blackfire updated the documentation : they said that is supported including 8.4 and 8.5

- *ev* is not available in PHP 8.1+
- *rdkafka* and *snmp* are not available in all `ARM64` images (build time is too long : it's possible to install manually as required)
- *blackfire* not supported yet in PHP8.4 : https://support.blackfire.platform.sh/hc/en-us/articles/4740726712082-PHP-versions-Platforms-support
- *blackfire* not supported yet in PHP8.4+ : https://support.blackfire.platform.sh/hc/en-us/articles/4740726712082-PHP-versions-Platforms-support
Copy link
Collaborator

Choose a reason for hiding this comment

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

@mistraloz
Copy link
Collaborator

mistraloz commented Dec 1, 2025

Thx a lot for your PR. Please may you check the CI and fix it ?
On Dockerfile.slim.apache:52
52 | >>> RUN apt update
53 | >>> && apt install -y --no-install-recommends
54 | >>> php${PHP_VERSION}-cli
55 | >>> php${PHP_VERSION}-curl
56 | >>> php${PHP_VERSION}-mbstring
57 | >>> php${PHP_VERSION}-opcache
58 | >>> php${PHP_VERSION}-readline
59 | >>> php${PHP_VERSION}-xml
60 | >>> php${PHP_VERSION}-zip \

From 8.5, opcache is a builtin feature so php${PHP_VERSION}-opcache is no more required.
I suggest something like that :

RUN OPCACHE_PACKAGE="" \
    && if [ "${PHP_VERSION}" != "8.5" ]; then \
        OPCACHE_PACKAGE="php${PHP_VERSION}-opcache"; \
    fi \
    \
    && apt update \
           php${PHP_VERSION}-zip \
           ${OPCACHE_PACKAGE} \

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