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

PHP 8 #79

Closed
faca5 opened this issue Dec 3, 2020 · 11 comments
Closed

PHP 8 #79

faca5 opened this issue Dec 3, 2020 · 11 comments

Comments

@faca5
Copy link

faca5 commented Dec 3, 2020

Hello.

I got errors:

Command:
make

Errors:

NON_BLOCKING_IO_php80/php7/memcache.c:43:2: error: initializer element is not constant

...
te: expected 'struct zend_object *' but argument is of type 'stru
...

warning: passing argument 2 of 'zend_read_property' from incompatible pointer type [enabled by 

Thank you.

@andypost
Copy link
Contributor

andypost commented Dec 6, 2020

Used to build just released 8.0 version and getting

php8 --ri memcache

memcache

memcache support => enabled
Version => 4.0.5.2
Revision => $Revision$

Directive => Local Value => Master Value
memcache.allow_failover => 1 => 1
memcache.max_failover_attempts => 20 => 20
memcache.default_port => 11211 => 11211
memcache.chunk_size => 32768 => 32768
memcache.protocol => ascii => ascii
memcache.hash_strategy => consistent => consistent
memcache.hash_function => crc32 => crc32
memcache.redundancy => 1 => 1
memcache.session_redundancy => 2 => 2
memcache.compress_threshold => 20000 => 20000
memcache.lock_timeout => 15 => 15
memcache.session_prefix_host_key => Off => Off
memcache.session_prefix_host_key_remove_www => On => On
memcache.session_prefix_host_key_remove_subdomain => Off => Off
memcache.session_prefix_static_key => no value => no value
memcache.session_save_path => no value => no value
memcache.prefix_host_key => Off => Off
memcache.prefix_host_key_remove_www => On => On
memcache.prefix_host_key_remove_subdomain => Off => Off
memcache.prefix_static_key => no value => no value

@andypost
Copy link
Contributor

andypost commented Dec 6, 2020

@tomassrnka
Copy link
Member

Hopefully resolved now, sorry about that

@andypost
Copy link
Contributor

andypost commented Dec 6, 2020

FYI packaged to Alpinelinux testing repo https://pkgs.alpinelinux.org/packages?name=php8-pecl-memcache&branch=edge it skips few tests because it tricky to run enough services

@faca5
Copy link
Author

faca5 commented Dec 7, 2020

Thank you.

@glensc
Copy link
Contributor

glensc commented Dec 27, 2021

Perhaps this should be closed due to the unhelpful issue title as the module supporting PHP 8.0 is already released.

@nono303
Copy link

nono303 commented Dec 28, 2021

Perhaps this should be closed due to the unhelpful issue title as the module supporting PHP 8.0 is already released.

https://github.com/nono303/PHP-memcache-dll for Windows binaries

@Jan-E
Copy link
Contributor

Jan-E commented Apr 30, 2023

i disagree that the PHP8 issue is completed, because win32/php_stdint.h is still there in the main branch. Of course you can patch it before building on Windows like @nono303 and I am doing, but that is not as it should be.

@tomassrnka
Copy link
Member

@Jan-E is this the patch that makes the extension build & work on windows? https://github.com/nono303/PHP-memcache-dll/blob/master/php82.patch

Unfortunately I have no idea how to test it, but if there is PR I can merge it and tag it with win support

@Jan-E
Copy link
Contributor

Jan-E commented Apr 30, 2023

I am not sure if that is complete. PHP 7.0 up until 7.4 still had win32/php_stdint,h on board, which worked fine. Better to use that than rely on a stdint.h provided by VC14 or VC15. I will test and supply a complete patch later. Might take some time, because I am now going to visit my mother in Rotterdam. Respectable age: she turms 96 on May 3rd.

@Jan-E
Copy link
Contributor

Jan-E commented May 2, 2023

#110 fixes building from at least PHP 7.2. I did not try PHP 7.0 and 7.1 yet.

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

6 participants