Project

Profile

Help

Feature #4842

open

PHP 8

Added by Peter Jan over 1 year ago. Updated 4 months ago.

Status:
New
Priority:
High
Category:
PHP Build
Start date:
2020-11-26
Due date:
% Done:

0%

Estimated time:
Found in version:
Fixed in version:
Platforms:

Description

PHP 8 has been released today, so inevitably someone has to create a ticket and ask if it becomes supported ;)

Is support for PHP8 planned for the next release?


Related issues

Related to SaxonC - Bug #5241: PHP API for Saxon/C HE for PHP version 8.NewCommunity Admin2022-01-26

Actions
Actions #1

Updated by Peter Jan over 1 year ago

Apologies; should be a feature request and not a bug report.

Actions #2

Updated by Michael Kay over 1 year ago

Perhaps you could be the first to try it and tell us if it already works?

With most platforms (Java, C#, etc) we don't explicitly test or support the product when a new version of the platform comes out, we just rely on the user community to tell us if there's a problem. Usually there isn't.

Actions #3

Updated by Peter Jan over 1 year ago

Ah! In that case, unfortunately it doesn't work. Because the final version is not yet available as a docker image, I tested it on php:8.0.0RC5-fpm. It was working on php:7.4.12-fpm.

Please find the full log here: https://pastebin.com/m3rHAsrL.

To reproduce, run the following Dockerfile:

FROM php:8.0.0RC5-fpm AS php

RUN set -eux; \
    apt-get update && apt-get install -y --no-install-recommends \
        unzip \
    ; \
    rm -rf /var/lib/apt/lists/*;

ARG saxon='libsaxon-HEC-setup64-v1.2.1'
RUN set -eux; \
    cd /tmp && curl -s -o ${saxon}.zip https://www.saxonica.com/saxon-c/${saxon}.zip; \
    unzip ${saxon}.zip; \
    ./${saxon} -batch -dest /tmp/saxon; \
    cp /tmp/saxon/libsaxonhec.so /usr/lib/; \
    cp -r /tmp/saxon/rt /usr/lib; \
    ldconfig; \
    cd /tmp/saxon/Saxon.C.API/; \
    sed -i 's/#define PHP_SAXON_EXTNAME  "Saxon\/C"/#define PHP_SAXON_EXTNAME  "SaxonC"/g' php_saxon.h; \
    phpize; \
    ./configure --enable-saxon; \
    make -j$(nproc); \
    make install; \
    docker-php-ext-enable saxon; \
    rm -rf /tmp/${saxon}.zip /tmp/${saxon} /tmp/saxon;
Actions #4

Updated by O'Neil Delpratt over 1 year ago

  • Tracker changed from Bug to Feature

Thanks for looking into this. I am not surprised it does not work. We will have investigate what changes need to be made to migrate our extension from PHP7 to PHP8.

Aside from PHP7 and PHP8 support, I am now wondering if we still need to support PHP5.

Actions #5

Updated by Peter Jan over 1 year ago

Well considering that PHP 5 went EOL almost 2 years ago :) I'm not sure if that would mean improvements can be made which were previously not possible because you had to support both versions? I'm not an extension developer so I really couldn't tell.

https://www.php.net/supported-versions.php

The build problems reported above look really similar to the issues reported here: https://github.com/docker-library/php/issues/571

But maybe you already know :)

Actions #6

Updated by Peter Jan over 1 year ago

I noticed in another issue you're mentioning Saxon/C 1.3; Is PHP 8 support planned for 1.3 as well? I'm asking because this extension is currently my last blocker for using PHP 8 and I would very much like to upgrade.

I found the PR's below where another extension is upgraded to PHP 8, maybe they're of use :).

https://github.com/php-decimal/ext-decimal/pull/53 https://github.com/php-decimal/ext-decimal/pull/54

Actions #7

Updated by O'Neil Delpratt over 1 year ago

  • Category set to PHP Build
  • Assignee set to O'Neil Delpratt
  • Priority changed from Low to Normal

Hi thanks for the reminder on this issue. Saxon/C 1.3 development is coming alone well, hopefully I can start looking at the PHP 8 support by the end of this week.

Actions #8

Updated by Peter Jan over 1 year ago

Awesome. Thank you!

Actions #9

Updated by Rein Baarsma 9 months ago

Any update? We'd like to update to PHP 8.0 and later this year to PHP 8.1

PHP 7.4 active support is ending in november..

Actions #10

Updated by O'Neil Delpratt 9 months ago

Hi, sorry I have not investigated this as yet. It is still on our to do list.

Actions #11

Updated by Alan Bachelin 6 months ago

Hi, have you been able to make progress on the update for compatibility with php 8? thanks in advance

Actions #12

Updated by O'Neil Delpratt 5 months ago

Hi, It is still on our todo list.

Actions #13

Updated by O'Neil Delpratt 4 months ago

  • Priority changed from Normal to High

PHP 8 did not make it in the SaxonC 11 release. But we hope to work on it for the next maintenance release of SaxonC.

Actions #14

Updated by O'Neil Delpratt 3 months ago

  • Related to Bug #5241: PHP API for Saxon/C HE for PHP version 8. added

Please register to edit this issue

Also available in: Atom PDF