aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/command/status_command.rs
diff options
context:
space:
mode:
Diffstat (limited to 'crates/shirabe/src/command/status_command.rs')
-rw-r--r--crates/shirabe/src/command/status_command.rs38
1 files changed, 15 insertions, 23 deletions
diff --git a/crates/shirabe/src/command/status_command.rs b/crates/shirabe/src/command/status_command.rs
index 182dec8..9f66adf 100644
--- a/crates/shirabe/src/command/status_command.rs
+++ b/crates/shirabe/src/command/status_command.rs
@@ -125,19 +125,18 @@ impl StatusCommand {
let target_dir = composer
.get_installation_manager()
.borrow_mut()
- .get_install_path(package.as_rc().borrow().as_package_interface());
+ .get_install_path(package.clone());
let target_dir = match target_dir {
Some(d) => d,
None => continue,
};
// TODO(phase-b): downloader borrow lifetime tied to dm.borrow() temporary; restructure later.
let dm_borrow = dm.borrow();
- let downloader: &dyn crate::downloader::DownloaderInterface = match dm_borrow
- .get_downloader_for_package(package.as_rc().borrow().as_package_interface())?
- {
- Some(d) => d,
- None => continue,
- };
+ let downloader: &dyn crate::downloader::DownloaderInterface =
+ match dm_borrow.get_downloader_for_package(package.clone())? {
+ Some(d) => d,
+ None => continue,
+ };
// TODO(phase-b): isinstance checks using ChangeReportInterface/VcsCapableDownloaderInterface/DvcsDownloaderInterface
if let Some(change_reporter) = downloader.as_change_report_interface() {
@@ -148,20 +147,16 @@ impl StatusCommand {
);
}
- if let Some(changes) = change_reporter.get_local_changes(
- package.as_rc().borrow().as_package_interface(),
- &target_dir,
- )? {
+ if let Some(changes) =
+ change_reporter.get_local_changes(package.clone(), &target_dir)?
+ {
errors.insert(target_dir.clone(), changes);
}
}
if let Some(vcs_downloader) = downloader.as_vcs_capable_downloader_interface() {
if vcs_downloader
- .get_vcs_reference(
- package.as_rc().borrow().as_package_interface(),
- target_dir.clone(),
- )
+ .get_vcs_reference(package.clone(), target_dir.clone())
.is_some()
{
let previous_ref = match package.get_installation_source().as_deref() {
@@ -170,10 +165,8 @@ impl StatusCommand {
_ => None,
};
- let current_version = guesser.guess_version(
- &dumper.dump(package.as_rc().borrow().as_package_interface()),
- &target_dir,
- )?;
+ let current_version =
+ guesser.guess_version(&dumper.dump(package.clone()), &target_dir)?;
if let (Some(prev_ref), Some(cur_version)) = (&previous_ref, &current_version) {
if cur_version.commit.as_deref() != Some(prev_ref.as_str())
@@ -207,10 +200,9 @@ impl StatusCommand {
}
if let Some(dvcs_downloader) = downloader.as_dvcs_downloader_interface() {
- if let Some(unpushed) = dvcs_downloader.get_unpushed_changes(
- package.as_rc().borrow().as_package_interface(),
- target_dir.clone(),
- ) {
+ if let Some(unpushed) =
+ dvcs_downloader.get_unpushed_changes(package.clone(), target_dir.clone())
+ {
unpushed_changes.insert(target_dir, unpushed);
}
}