Skip to content

pdo_pgsql cannot build on PHP 8.4 #13745

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

Closed
mvorisek opened this issue Mar 18, 2024 · 5 comments
Closed

pdo_pgsql cannot build on PHP 8.4 #13745

mvorisek opened this issue Mar 18, 2024 · 5 comments

Comments

@mvorisek
Copy link
Contributor

mvorisek commented Mar 18, 2024

Description

...
 cc -I. -I/usr/src/php/ext/pdo_pgsql -I/usr/src/php/ext/pdo_pgsql/main -I/usr/src/php/ext/pdo_pgsql -I/usr/local/include/php -I/usr/local/include/php/main -I/usr/local/include/php/TSRM -I/usr/local/include/php/Zend -I/usr/local/include/php/ext -I/usr/local/include/php/ext/date/lib -I/usr/include/postgresql -fstack-protector-strong -fpic -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -DHAVE_CONFIG_H -fstack-protector-strong -fpic -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -I/usr/local/include/php/ext -DZEND_COMPILE_DL_EXT=1 -c /usr/src/php/ext/pdo_pgsql/pgsql_statement.c -MMD -MF pgsql_statement.dep -MT pgsql_statement.lo  -fPIC -DPIC -o .libs/pgsql_statement.o
 cc -I. -I/usr/src/php/ext/pdo_pgsql -I/usr/src/php/ext/pdo_pgsql/main -I/usr/src/php/ext/pdo_pgsql -I/usr/local/include/php -I/usr/local/include/php/main -I/usr/local/include/php/TSRM -I/usr/local/include/php/Zend -I/usr/local/include/php/ext -I/usr/local/include/php/ext/date/lib -I/usr/include/postgresql -fstack-protector-strong -fpic -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -DHAVE_CONFIG_H -fstack-protector-strong -fpic -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -I/usr/local/include/php/ext -DZEND_COMPILE_DL_EXT=1 -c /usr/src/php/ext/pdo_pgsql/pdo_pgsql.c -MMD -MF pdo_pgsql.dep -MT pdo_pgsql.lo  -fPIC -DPIC -o .libs/pdo_pgsql.o
 cc -I. -I/usr/src/php/ext/pdo_pgsql -I/usr/src/php/ext/pdo_pgsql/main -I/usr/src/php/ext/pdo_pgsql -I/usr/local/include/php -I/usr/local/include/php/main -I/usr/local/include/php/TSRM -I/usr/local/include/php/Zend -I/usr/local/include/php/ext -I/usr/local/include/php/ext/date/lib -I/usr/include/postgresql -fstack-protector-strong -fpic -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -DHAVE_CONFIG_H -fstack-protector-strong -fpic -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -I/usr/local/include/php/ext -DZEND_COMPILE_DL_EXT=1 -c /usr/src/php/ext/pdo_pgsql/pgsql_driver.c -MMD -MF pgsql_driver.dep -MT pgsql_driver.lo  -fPIC -DPIC -o .libs/pgsql_driver.o
/usr/src/php/ext/pdo_pgsql/pdo_pgsql.c:25:10: fatal error: pdo/php_pdo_int.h: No such file or directory
   25 | #include "pdo/php_pdo_int.h"
      |          ^~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:209: pdo_pgsql.lo] Error 1

full CI log: https://github.com/mvorisek/docker-php-extension-installer/actions/runs/8330250368/job/22794461675#step:4:148

PHP Version

master

Operating System

Debian (on Alpine the build passes)

@devnexen
Copy link
Member

seems not needed anymore inclusion ? will have a look

@devnexen
Copy link
Member

@mvorisek What would be the ./configure part of this ?

@mvorisek
Copy link
Contributor Author

I install it using https://github.com/mlocati/docker-php-extension-installer/blob/2.2.5/install-php-extensions which does nothing special for pdo_pgsql, so it should be reproducible using https://github.com/docker-library/php/tree/master/8.3/bookworm/cli . It seems nothing special is done there too. The only difference vs. the php-src CI should be the ext is build separately.

@devnexen
Copy link
Member

can you confirm something for me ? replacing the failing header inclusion by pdo/php_pdo_error.h

devnexen added a commit to devnexen/php-src that referenced this issue Mar 18, 2024
pdo/php_pdo_int.h is not part of the PDO's headers to install.
@mvorisek
Copy link
Contributor Author

#13749 seems to fix this issue, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants