Skip to content

Add zend_worklist.h to PHP_INSTALL_HEADERS - #12571

Merged
iluuu1994 merged 1 commit into
php:masterfrom
iluuu1994:worklist-install-headers
Oct 30, 2023
Merged

Add zend_worklist.h to PHP_INSTALL_HEADERS#12571
iluuu1994 merged 1 commit into
php:masterfrom
iluuu1994:worklist-install-headers

Conversation

@iluuu1994

Copy link
Copy Markdown
Member

Fixes GH-12565

@devnexen

Copy link
Copy Markdown
Member

I would say LGTM, but not sure that is really the issue in the aforementioned ticket.
What I mean is maybe, since there was no -I/usr/src/php/Zend, the build pointed to an existing php installation rather than the PHP's source content, if that makes sense ?
But your change is ok, do not see any harm.

@iluuu1994

Copy link
Copy Markdown
Member Author

@devnexen I don't know exactly how the Docker image is set up, but I could imagine PHP being built, installed, and then opcache being built against the installed sources. This would be missing this header. This header was added to zend_jit.c as of a few days ago, with the new JIT.

@devnexen devnexen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

fair enough

@mvorisek

Copy link
Copy Markdown
Contributor

@devnexen I don't know exactly how the Docker image is set up, but I could imagine PHP being built, installed, and then opcache being built against the installed sources. This would be missing this header. This header was added to zend_jit.c as of a few days ago, with the new JIT.

yes, this is exactly what I did and what https://hub.docker.com/_/php (official PHP docker image) does

@iluuu1994
iluuu1994 merged commit 2efe366 into php:master Oct 30, 2023
@iluuu1994

Copy link
Copy Markdown
Member Author

@mvorisek Can you confirm that this fixes your build?

@mvorisek

Copy link
Copy Markdown
Contributor

@mvorisek Can you confirm that this fixes your build?

I can confirm the issue is gone, thank you both ❤️

adrian-enspired pushed a commit to adrian-enspired/php-src that referenced this pull request Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make of opcache ext is failing on master

3 participants