Skip to content

Conversation

@gcavanunez
Copy link
Contributor

@gcavanunez gcavanunez commented Oct 8, 2025

Changes

Adds the pdo_sqlite & pdo_pgsql to the matrix release config since pdo_mysql was already included, however for sqlite and pgsql these extensions seemed to be missing.

Been really liking the project! it's super handy thanks for assembling this ya'll

@crazywhalecc
Copy link
Member

crazywhalecc commented Oct 12, 2025

Thank you for contributing this.

The common combination looks good to add pdo_pgsql, but it should also add pgsql extension at the same time.

And the bulk and gnu-bulk has swoole-hook-pgsql and swoole-hook-sqlite has included, it provides pdo_pgsql and pdo_sqlite same functions.

@gcavanunez
Copy link
Contributor Author

gcavanunez commented Oct 16, 2025

Added the pgsql to the common, but when checking the extensions loaded was when I noticed that the pdo_sqlite and pod_pgsql weren't being loaded, though it sounds like perhaps that because of the swoole-hook-* or would this be perhaps a bug?

Loaded Extensions

``` echo var_dump(get_loaded_extensions()); ``` ``` array(62) { [0]=> string(4) "Core" [1]=> string(4) "date" [2]=> string(6) "libxml" [3]=> string(7) "openssl" [4]=> string(4) "pcre" [5]=> string(7) "sqlite3" [6]=> string(4) "zlib" [7]=> string(4) "apcu" [8]=> string(6) "bcmath" [9]=> string(3) "bz2" [10]=> string(8) "calendar" [11]=> string(5) "ctype" [12]=> string(4) "curl" [13]=> string(3) "dba" [14]=> string(3) "dom" [15]=> string(7) "sockets" [16]=> string(5) "event" [17]=> string(4) "hash" [18]=> string(8) "fileinfo" [19]=> string(6) "filter" [20]=> string(3) "ftp" [21]=> string(2) "gd" [22]=> string(3) "gmp" [23]=> string(4) "json" [24]=> string(5) "iconv" [25]=> string(3) "SPL" [26]=> string(7) "session" [27]=> string(4) "intl" [28]=> string(8) "standard" [29]=> string(8) "mbstring" [30]=> string(7) "mysqlnd" [31]=> string(6) "mysqli" [32]=> string(13) "opentelemetry" [33]=> string(5) "pcntl" [34]=> string(4) "exif" [35]=> string(4) "imap" [36]=> string(3) "PDO" [37]=> string(9) "pdo_mysql" [38]=> string(5) "pgsql" [39]=> string(4) "Phar" [40]=> string(5) "posix" [41]=> string(8) "protobuf" [42]=> string(6) "random" [43]=> string(8) "readline" [44]=> string(5) "redis" [45]=> string(10) "Reflection" [46]=> string(7) "imagick" [47]=> string(5) "shmop" [48]=> string(9) "SimpleXML" [49]=> string(4) "soap" [50]=> string(6) "sodium" [51]=> string(6) "swoole" [52]=> string(7) "sysvmsg" [53]=> string(7) "sysvsem" [54]=> string(7) "sysvshm" [55]=> string(9) "tokenizer" [56]=> string(3) "xml" [57]=> string(9) "xmlreader" [58]=> string(9) "xmlwriter" [59]=> string(3) "xsl" [60]=> string(3) "zip" [61]=> string(12) "Zend OPcache" } ```

@crazywhalecc
Copy link
Member

crazywhalecc commented Oct 24, 2025

That's because swoole-hook-pgsql provides the same function as pdo_pgsql but not registering the new extension, the extension list will not show it as expected.

@crazywhalecc crazywhalecc merged commit 23621b0 into static-php:master Oct 24, 2025
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