Stdlib: version_compare() — shorter operand must compare like Zend (ext/standard/versioning.c)

Problem

version_compare() in ext/standard/VmInfo.php diverges from Zend when one operand has fewer dotted segments (e.g. 8.2 vs 8.2.31).

php-src reference

php/php-src ext/standard/versioning.c — php_version_compare(), php_canonicalize_version()

Repro

test/tmp/repro-version-compare-partial.php:

<?php
declare(strict_types=1);
var_export(version_compare('8.2.31', '8.2', '>='));
echo PHP_EOL;

./script/docker-exec.sh -- bash -lc 'php test/tmp/repro-version-compare-partial.php; php bin/vm.php test/tmp/repro-version-compare-partial.php'

Case	Zend	This compiler
`version_compare('8.2.31', '8.2', '>=')`	`true`	`false`

Scope

Module	Path
VM	`ext/standard/VmInfo.php` (`phpVersionCompare` / `canonicalizeVersion`)
JIT	`lib/JIT/Builtin/` / `__compiler_version_compare` if duplicated
Tests	`test/compliance/cases/stdlib/version_compare_partial.phpt`

Done when

Repro matches Zend on VM (and JIT if lowered)
./script/ci-fast.sh --filter version_compare_partial green

Stdlib: version_compare() — invalid $operator must throw ValueError (ext/standard/versioning.c parity) #4319 — invalid $operator must be ValueError
Stdlib: version_compare() / extension_loaded() / get_loaded_extensions() (ext/standard parity) #3204 — initial version_compare() subset

claim: PHP Compiler STD Lib — starting this run

Fix version_compare() canonicalize for dotted segments (#4796) (#5327)

Merged via #5327

Root cause: canonicalizeVersion() treated . as a digit/non-digit boundary, producing 8..2..31 instead of 8.2.31.

Fix: Match php-src versioning.c isdig/isndig — exclude . from the transition branch in VmInfo.php and phpc_info.c.

Verification:

php test/tmp/repro-version-compare-partial.php          → true
php bin/vm.php test/tmp/repro-version-compare-partial.php → true
vendor/bin/phpunit --filter version_compare_partial     → OK (2 tests)

Cleanup: deleting leftover test/tmp/ scratch repro for this closed issue — same coverage lives under test/compliance/cases/ (see PR chore/cleanup-stale-test-tmp-repros).

Cleanup follow-up: merged #25413 — test/tmp scratch repro removed.

Chore: drop stale test/tmp repros superseded by compliance (#4812/#4810/

Navigation Menu

Repository navigation

Stdlib: version_compare() — shorter operand must compare like Zend (ext/standard/versioning.c) #4796

Category

Problem

php-src reference

Repro

Scope

Done when

Related

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Stdlib: version_compare() — shorter operand must compare like Zend (ext/standard/versioning.c) #4796

Description

Category

Problem

php-src reference

Repro

Scope

Done when

Related

Activity

PurHur commented on Jun 3, 2026

PurHur commented on Jun 3, 2026

PurHur commented on Jul 30, 2026

PurHur commented on Jul 30, 2026

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions