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

[BUG]: Failure to Install with PECL #16476

Open
aaugusto-kelmar opened this issue Dec 19, 2023 · 3 comments
Open

[BUG]: Failure to Install with PECL #16476

aaugusto-kelmar opened this issue Dec 19, 2023 · 3 comments
Labels
bug A bug report status: unverified Unverified

Comments

@aaugusto-kelmar
Copy link

aaugusto-kelmar commented Dec 19, 2023

Describe the bug
When installing via PECL (and also when trying to install with the install script in the build directory), the install gets stuck in a loop spitting out the following error:

/usr/include/php/20210902/Zend/zend_API.h:1984:76: note: expected 'zend_string **' {aka 'struct _zend_string **'} but argument is of type 'zval *' {aka 'struct _zval_struct *'}
 1984 | static zend_always_inline bool zend_parse_arg_str(zval *arg, zend_string **dest, bool check_null, uint32_t arg_num)
      |                                                              ~~~~~~~~~~~~~~^~~~
In file included from /usr/include/php/20210902/Zend/zend_types.h:25,
                 from /usr/include/php/20210902/Zend/zend.h:27,
                 from /usr/include/php/20210902/main/php.h:32,
                 from /tmp/pear/temp/phalcon/phalcon.zep.c:46:
/tmp/pear/temp/phalcon/phalcon.zep.c: In function 'zim_Phalcon_Db_Dialect_Mysql_createTable':
/usr/include/php/20210902/Zend/zend_API.h:1815:58: warning: passing argument 2 of 'zend_parse_arg_str' from incompatible pointer type [-Wincompatible-pointer-types]
 1815 |                 if (UNEXPECTED(!zend_parse_arg_str(_arg, &dest, check_null, _i))) { \
/usr/include/php/20210902/Zend/zend_portability.h:338:52: note: in definition of macro 'UNEXPECTED'
  338 | # define UNEXPECTED(condition) __builtin_expect(!!(condition), 0)
      |                                                    ^~~~~~~~~
/usr/include/php/20210902/Zend/zend_API.h:1822:9: note: in expansion of macro 'Z_PARAM_STR_EX'
 1822 |         Z_PARAM_STR_EX(dest, 0, 0)
      |         ^~~~~~~~~~~~~~
/tmp/pear/temp/phalcon/phalcon.zep.c:68603:17: note: in expansion of macro 'Z_PARAM_STR'
68603 |                 Z_PARAM_STR(tableName)
      |                 ^~~~~~~~~~~
In file included from /usr/include/php/20210902/main/php.h:36,
                 from /tmp/pear/temp/phalcon/phalcon.zep.c:46:
/usr/include/php/20210902/Zend/zend_API.h:1984:76: note: expected 'zend_string **' {aka 'struct _zend_string **'} but argument is of type 'zval *' {aka 'struct _zval_struct *'}
 1984 | static zend_always_inline bool zend_parse_arg_str(zval *arg, zend_string **dest, bool check_null, uint32_t arg_num)

To Reproduce

pecl channel-update pecl.php.net
pecl install phalcon

Expected behavior
Phalcon installed successfully

Details

  • Phalcon version: v5.4.0, also tried v5.3.0 and using the 5.0.x branch
  • PHP Version: 8.1.2-1ubuntu2.14
  • Operating System: ubuntu
  • Installation type: both compiling from source and installing via package manager
  • Zephir version (if any): none

Additional context
Install was working last week but as of Friday it stopped working and would encounter this error.

@aaugusto-kelmar aaugusto-kelmar added bug A bug report status: unverified Unverified labels Dec 19, 2023
@s-ohnishi
Copy link

Maybe the machine you're using doesn't have enough specs (especially memory)?
I am using Phalcon in a Docker container, and have had the experience of completing compilation by increasing the memory allocated to WSL2 (as a result, increasing the container's memory).
This post states that at least 3Gb of memory is required.

@kgrammer
Copy link

@niden and I were working on a PHP 8.3 build issue and I confirmed that PECL does not work on a small server. I was trying to install with PECL on a 1CPU/2GB server and PECL was failing. When I upgraded to a 2CPU/4GB server, PECL installed Phalcon as expected.

@andypost
Copy link

andypost commented Feb 9, 2024

Confirm that build log is full of warnings like summary notes, building it for Alpinelinux http://build.alpinelinux.org/buildlogs/build-edge-x86_64/testing/php81-pecl-phalcon/php81-pecl-phalcon-5.6.0-r0.log

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug report status: unverified Unverified
Projects
None yet
Development

No branches or pull requests

4 participants