Bug #74082 [Opn->Csd]: strncmp_variation6 test fails on AArch64
| From: | cmb@php.net | Date: | Thu, 29 Nov 2018 22:51:49 +0000 |
| Subject: | Bug #74082 [Opn->Csd]: strncmp_variation6 test fails on AArch64 | ||
| References: | 1 | Groups: | php.bugs php.qa |
| Request: | Send a blank email to php-bugs+get-218218@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=74082&edit=1
ID: 74082
Updated by: cmb@php.net
Reported by: edward dot vielmetti at gmail dot com
Summary: strncmp_variation6 test fails on AArch64
-Status: Open
+Status: Closed
Type: Bug
Package: Testing related
Operating System: Ubuntu 16.04 LTS on aarch64
PHP Version: 7.1.1
-Assigned To:
+Assigned To: cmb
Block user comment: N
Private report: N
New Comment:
As discussed on PR #3689[1], the pointless test has been dropped
altogether[2].
[1] <https://github.com/php/php-src/pull/3689>
[2] <http://git.php.net/?p=php-src.git;a=commit;h=ea1ea33bd5d967e9c04046c1af869e4e8b53116e>
Previous Comments:
------------------------------------------------------------------------
[2018-07-16 13:12:08] cmb@php.net
This test[1] confuses binary strings and binary numbers, and
should better be rewritten or dropped, anyway.
Furthermore, and this is might be the reason for the reported test
failure, the test assumes that strncmp() would return exactly 1,
if the first string is greater than the second, but strncmp() only
guarantees that a positive integer is returned in this case.
[1] <https://github.com/php/php-src/blob/php-7.1.1/ext/standard/tests/strings/strncmp_variation6.phpt>
------------------------------------------------------------------------
[2017-02-10 21:42:05] edward dot vielmetti at gmail dot com
root@docker-build-test:/mnt/src/php/php-7.1.1/ext/standard/tests/strings# more strncmp*diff
774+ int(128)
774- int(1)
------------------------------------------------------------------------
[2017-02-10 21:19:41] nikic@php.net
What's the diff? (ext/standard/tests/strings/strncmp_variation6.diff)
------------------------------------------------------------------------
[2017-02-10 21:01:20] edward dot vielmetti at gmail dot com
Description:
------------
FAILED TEST SUMMARY
---------------------------------------------------------------------
Test strncmp() function : usage variations - binary safe(binary values)
[ext/standard/tests/strings/strncmp_variation6.phpt]
"make test" fails this one test on aarch64 (ARMv8, arm64) on Packet 2A 96-core system.
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=74082&edit=1