Skip to content

32bits failed tests #318

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

Closed
remicollet opened this issue Feb 9, 2017 · 2 comments
Closed

32bits failed tests #318

remicollet opened this issue Feb 9, 2017 · 2 comments
Milestone

Comments

@remicollet
Copy link
Collaborator

remicollet commented Feb 9, 2017

On x86_64, test suite passes

On i386:

TEST 35/81 [tests/gh_90.phpt]
========DIFF========
001+ Warning: mt_rand() expects parameter 2 to be integer, float given in /builddir/build/BUILD/php-pecl-memcached-3.0.1/NTS/tests/gh_90.php on line 10
========DONE========
FAIL Test for GH #90 [tests/gh_90.phpt] 

TEST 37/81 [tests/incrdecr.phpt]
========DIFF========
020+ int(1)
020- int(4294967297)
========DONE========
FAIL Memcached::increment() Memcached::decrement() [tests/incrdecr.phpt] 

TEST 38/81 [tests/incrdecr_bykey.phpt]
========DIFF========
017+ int(1)
017- int(4294967297)
========DONE========

Number of tests :   81                78
Tests skipped   :    3 (  3.7%) --------
Tests warned    :    1 (  1.2%) (  1.3%)
Tests failed    :    3 (  3.7%) (  3.8%)
Expected fail   :    0 (  0.0%) (  0.0%)
Tests passed    :   74 ( 91.4%) ( 94.9%)

Will try to propose a fix

@sodabrew sodabrew added this to the 3.0.2 milestone Feb 9, 2017
@sodabrew
Copy link
Contributor

sodabrew commented Feb 9, 2017

Tagging this issue for 3.0.2 release. If it's preventing you from making distribution builds I'll be sure to cut the release as soon as we sort out the #319 #320 #321 PRs

@sodabrew
Copy link
Contributor

Resolved by #319 to remove the mandatory 64-bit tests and #321 to split them out as separate 64-bit-only tests when 64-bits are available.

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