diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-05-25 00:58:20 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-05-25 00:58:36 +0900 |
| commit | 1921f173ea219cb4b25847294d2d3fa465550fbb (patch) | |
| tree | 0d30486a2cb9a0c106e5d5827be3f655c60cd871 /crates/shirabe/src/command/check_platform_reqs_command.rs | |
| parent | dbdecaf5a1c54a876b7ee0153d58dd39b1080f97 (diff) | |
| download | php-shirabe-1921f173ea219cb4b25847294d2d3fa465550fbb.tar.gz php-shirabe-1921f173ea219cb4b25847294d2d3fa465550fbb.tar.zst php-shirabe-1921f173ea219cb4b25847294d2d3fa465550fbb.zip | |
refactor(package): introduce Rc<RefCell<_>> handles for packages
PHP packages have reference semantics, so introduce shared-ownership
handles over an AnyPackage enum (PackageInterfaceHandle and friends)
and replace Box<dyn PackageInterface> throughout.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/command/check_platform_reqs_command.rs')
| -rw-r--r-- | crates/shirabe/src/command/check_platform_reqs_command.rs | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/crates/shirabe/src/command/check_platform_reqs_command.rs b/crates/shirabe/src/command/check_platform_reqs_command.rs index 74ecfc3..5d886e7 100644 --- a/crates/shirabe/src/command/check_platform_reqs_command.rs +++ b/crates/shirabe/src/command/check_platform_reqs_command.rs @@ -113,7 +113,7 @@ impl CheckPlatformReqsCommand { } } - let root_pkg_repo = RootPackageRepository::new(composer.get_package().clone_box()); + let root_pkg_repo = RootPackageRepository::new(composer.get_package().clone()); let installed_repo = InstalledRepository::new(vec![installed_repo_base, Box::new(root_pkg_repo)]); @@ -148,7 +148,7 @@ impl CheckPlatformReqsCommand { let mut req_results: Vec<CheckResult> = vec![]; 'candidates: for candidate in &candidates { let candidate_constraint: Option<AnyConstraint> = - if candidate.get_name() == require { + if candidate.get_name() == *require { let c = SimpleConstraint::new( "=".to_string(), candidate.get_version().to_string(), @@ -178,7 +178,7 @@ impl CheckPlatformReqsCommand { for link in links { if !link.get_constraint().matches(&candidate_constraint) { req_results.push(CheckResult { - platform_package: if candidate.get_name() == require { + platform_package: if candidate.get_name() == *require { candidate.get_pretty_name().to_string() } else { require.clone() @@ -186,7 +186,7 @@ impl CheckPlatformReqsCommand { version: candidate_constraint.get_pretty_string().to_string(), link: Some(link.clone()), status: "<error>failed</error>".to_string(), - provider: if candidate.get_name() == require { + provider: if candidate.get_name() == *require { String::new() } else { format!( @@ -200,7 +200,7 @@ impl CheckPlatformReqsCommand { } results.push(CheckResult { - platform_package: if candidate.get_name() == require { + platform_package: if candidate.get_name() == *require { candidate.get_pretty_name().to_string() } else { require.clone() @@ -208,7 +208,7 @@ impl CheckPlatformReqsCommand { version: candidate_constraint.get_pretty_string().to_string(), link: None, status: "<info>success</info>".to_string(), - provider: if candidate.get_name() == require { + provider: if candidate.get_name() == *require { String::new() } else { format!( |