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 7.2 compatibility #358

Closed
remicollet opened this issue Jun 24, 2017 · 5 comments
Closed

php 7.2 compatibility #358

remicollet opened this issue Jun 24, 2017 · 5 comments
Milestone

Comments

@remicollet
Copy link
Collaborator

remicollet commented Jun 24, 2017

Yes, early report ;)

Build seems ok, but test suite fails

TEST 3/83 [tests/append.phpt]
========DIFF========
002+ 
002- %s: cannot append/prepend with compression turned on
========DONE========
FAIL Memcached::append() [tests/append.phpt] 
TEST 4/83 [tests/bad_construct.phpt]
========DIFF========
002+ 
003+ Notice: Undefined variable: php_errormsg in /dev/shm/BUILD/php72-php-pecl-memcached-3.0.3/NTS/tests/bad_construct.php on line 4
004+ 
002- Memcached::__construct() expects parameter 1 to be string, object given
========DONE========
FAIL Memcached construct with bad arguments [tests/bad_construct.phpt] 
TEST 36/83 [tests/incrdecr.phpt]
========DIFF========
015+ 
015- Memcached::increment(): offset cannot be a negative value
017+ 
017- Memcached::decrement(): offset cannot be a negative value
========DONE========
FAIL Memcached::increment() Memcached::decrement() [tests/incrdecr.phpt] 
TEST 38/83 [tests/incrdecr_bykey.phpt]
========DIFF========
012+ 
012- Memcached::incrementByKey(): offset cannot be a negative value
014+ 
014- Memcached::decrementByKey(): offset cannot be a negative value
========DONE========
FAIL Memcached::incrementByKey() Memcached::decrementByKey() [tests/incrdecr_bykey.phpt] 
TEST 59/83 [tests/session_badconf_servers.phpt]
========DIFF========
002+ session_start(): Failed to initialize storage module: memcached (path: )
========DONE========
FAIL Session bad configurations, invalid save path (server list) [tests/session_badconf_servers.phpt] 


Tests failed    :    5 (  6.0%) (  6.3%)
Tests passed    :   74 ( 89.2%) ( 93.7%)

@sodabrew
Copy link
Contributor

sodabrew commented Jul 8, 2017

I've opened #359 but it's blocked on travis-ci/travis-ci#7989 to confirm if the changes are complete for 7.2.

@sodabrew sodabrew added this to the 3.1.0 milestone Jul 8, 2017
@sodabrew
Copy link
Contributor

sodabrew commented Jul 8, 2017

Resolved by 43bbf00

@sodabrew sodabrew closed this as completed Jul 8, 2017
@remicollet
Copy link
Collaborator Author

Thanks

Still 1 failed test:

TEST 59/83 [tests/session_badconf_servers.phpt]
========DIFF========
002+ session_start(): Failed to initialize storage module: memcached (path: )
========DONE========
FAIL Session bad configurations, invalid save path (server list) [tests/session_badconf_servers.phpt] 

Only an additional error message

@sodabrew sodabrew modified the milestones: 3.0.4, 3.1.0 Jul 11, 2017
@sodabrew
Copy link
Contributor

Fixed the session test in 39bfab0 it was a text-only change not a functional change.

@remicollet
Copy link
Collaborator Author

Thanks

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

2 participants