Skip to content
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.

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

Cannot build with php-src master/8.3 #582

Closed
mvorisek opened this issue Feb 1, 2023 · 4 comments
Closed

Cannot build with php-src master/8.3 #582

mvorisek opened this issue Feb 1, 2023 · 4 comments

Comments

@mvorisek
Copy link

mvorisek commented Feb 1, 2023

CI failure: https://github.com/mvorisek/image-php/actions/runs/4060500396/jobs/6989655041#step:13:8228

maybe related with #423

maybe caused by php/php-src@1fbe855 /cc @bwoebi

@Danack
Copy link
Collaborator

Danack commented Feb 1, 2023

Please take the time to report what the issue is in a bug report, not just link to it.

That webpage isn't displaying correctly for me, and the Imagick repo builds correctly for me against php-src latest.

@mvorisek
Copy link
Author

mvorisek commented Feb 1, 2023

...
PHP-Parser-4.15.1/lib/PhpParser/PrettyPrinter/
PHP-Parser-4.15.1/lib/PhpParser/PrettyPrinter/Standard.php
PHP-Parser-4.15.1/lib/PhpParser/PrettyPrinterAbstract.php
Saved /tmp/pear/temp/imagick/ImagickPixel_arginfo.h
In /tmp/pear/temp/imagick/Imagick.stub.php:
Unterminated preprocessor conditions
make: *** [Makefile:196: /tmp/pear/temp/imagick/Imagick_arginfo.h] Error 1
make: *** Waiting for unfinished jobs....
Installing header files:          /tmp/pear/temp/pear-build-defaultuserlNGJMa/install-imagick-3.7.0/usr/local/include/php/
ERROR: `make -j2 INSTALL_ROOT="/tmp/pear/temp/pear-build-defaultuserlNGJMa/install-imagick-3.7.0" install' failed
...

@Danack
Copy link
Collaborator

Danack commented Feb 1, 2023

I don't get that error.

Can you say what you're doing to get it?

@Danack
Copy link
Collaborator

Danack commented Feb 10, 2023

I can't investigate anything I can't reproduce. btw you probably don't need to be regenerating the arginfo files, so I have no idea what you are seeing.

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

No branches or pull requests

2 participants