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.

Sign up for GitHub

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

Jump to bottom

[0.12.7] undefined symbol ZEPHIR_NULL #1946

Closed
remicollet opened this issue Oct 4, 2019 · 2 comments
Closed

[0.12.7] undefined symbol ZEPHIR_NULL #1946

remicollet opened this issue Oct 4, 2019 · 2 comments
Labels
bug

Comments

@remicollet
Copy link
Contributor

Trying to run test suite.

PHP Warning: PHP Startup: Unable to load dynamic library '/dev/shm/BUILD/zephir-1c107e0c8966233f0d35156419e2c97729dcc4ce/ext/modules/test.so' (tried: /dev/shm/BUILD/zephir-1c107e0c8966233f0d35156419e2c97729dcc4ce/ext/modules/test.so (/dev/shm/BUILD/zephir-1c107e0c8966233f0d35156419e2c97729dcc4ce/ext/modules/test.so: undefined symbol: ZEPHIR_NULL), /usr/lib64/php/modules//dev/shm/BUILD/zephir-1c107e0c8966233f0d35156419e2c97729dcc4ce/ext/modules/test.so.so (/usr/lib64/php/modules//dev/shm/BUILD/zephir-1c107e0c8966233f0d35156419e2c97729
@remicollet
Copy link
Contributor Author

sed -e s'/ZEPHIR_NULL/ZVAL_NULL/' -i kernels/ZendEngine3/string.c seems enough

@remicollet remicollet mentioned this issue Oct 4, 2019
Merged
sergeyklay added a commit that referenced this issue Oct 4, 2019
@sergeyklay
Merge pull request #1947 from remicollet/patch-1
cb4a939 
Fix #1946 use ZVAL_NULL
@sergeyklay sergeyklay added the bug label Oct 4, 2019
@sergeyklay
Copy link
Contributor

I'm sorry, I introduced this. I hope it fixed now. Thank you again!

dreamsxin pushed a commit to dreamsxin/zephir that referenced this issue Nov 6, 2019
@remicollet @dreamsxin
Fix zephir-lang#1946 use ZVAL_NULL
81c83d9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@remicollet @sergeyklay