aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-08-09 07:36:40 +0900
committernsfisis <nsfisis@gmail.com>2026-08-09 07:36:40 +0900
commit627aaead32dcf38bed255012fcf7965d9553f52e (patch)
tree1dd58c19edb8b7d37aa30ac3593b8e1f608b8c32 /crates/shirabe
parentf1d65c337c63b6aa3582aabc64bfbbb2f949127c (diff)
downloadphp-shirabe-627aaead32dcf38bed255012fcf7965d9553f52e.tar.gz
php-shirabe-627aaead32dcf38bed255012fcf7965d9553f52e.tar.zst
php-shirabe-627aaead32dcf38bed255012fcf7965d9553f52e.zip
fix(dependency-resolver): query the PHP runtime for the php version
Problem reported the platform php version from the shim's compile-time PHP_VERSION constant, and the shim's phpversion() hit todo!() for any extension. Go through the RPC bridge instead, as PHP's Composer\Platform\Runtime does, and drop the shim function that now has no callers. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe')
-rw-r--r--crates/shirabe/src/dependency_resolver/problem.rs8
1 files changed, 3 insertions, 5 deletions
diff --git a/crates/shirabe/src/dependency_resolver/problem.rs b/crates/shirabe/src/dependency_resolver/problem.rs
index 3b9feba5..63cbc6fb 100644
--- a/crates/shirabe/src/dependency_resolver/problem.rs
+++ b/crates/shirabe/src/dependency_resolver/problem.rs
@@ -14,8 +14,8 @@ use shirabe_external_packages::composer::pcre::{CaptureKey, Preg};
use shirabe_external_packages::symfony::console::formatter::OutputFormatter;
use shirabe_php_shim::{
CmpOp, LogicException, PhpMixed, defined, extension_loaded, implode, loosely_compare,
- php_regex, phpversion, spl_object_hash, sprintf, str_replace, str_starts_with, stripos, strpos,
- strtolower, substr, substr_count, version_compare,
+ php_regex, spl_object_hash, sprintf, str_replace, str_starts_with, stripos, strpos, strtolower,
+ substr, substr_count, version_compare,
};
use shirabe_semver::constraint::AnyConstraint;
use shirabe_semver::constraint::MultiConstraint;
@@ -389,7 +389,7 @@ impl Problem {
let version = Self::get_platform_package_version(
pool,
package_name,
- &phpversion("").unwrap_or_default(),
+ &shirabe_php_rpc::phpversion("").unwrap_or_default(),
);
let msg = format!(
@@ -456,8 +456,6 @@ impl Problem {
Self::constraint_to_text(constraint)
);
- // Per-extension version info can't be known statically; query the real PHP
- // runtime via the RPC bridge, as PHP's Composer\Platform\Runtime does.
let runtime_version = shirabe_php_rpc::phpversion(&ext);
let effective_version = match runtime_version {
None => "0".to_string(),