Skip to content

Commit fd07629

Browse files
committed
Improve failure handling when JDK is gone completely
1 parent fbb25c1 commit fd07629

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

src/command/update.rs

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,16 +75,23 @@ impl JpreCommand for UpdateInstalled {
7575
let full_version = match JDK_MANAGER.get_full_version(&jdk) {
7676
Ok(full_version) => full_version,
7777
Err(err) => {
78-
warn!("Failed to get full version for {}: {}", jdk, err);
78+
warn!("Failed to get full version for {}: {:?}", jdk, err);
7979
continue;
8080
}
8181
};
8282

8383
if let Some(full_version) = full_version {
84-
let (list_info, _) = FOOJAY_API
84+
let latest_info_result = FOOJAY_API
8585
.get_latest_package_info_using_priority(&context.config, &jdk)
8686
.change_context(JpreError::Unexpected)
87-
.attach_printable("Failed to get latest package info")?;
87+
.attach_printable("Failed to get latest package info");
88+
let (list_info, _) = match latest_info_result {
89+
Ok(info) => info,
90+
Err(err) => {
91+
warn!("Failed to get latest package info for {}: {:?}", jdk, err);
92+
continue;
93+
}
94+
};
8895
let latest = list_info.java_version;
8996
if latest.compare(&full_version) == std::cmp::Ordering::Greater {
9097
eprintln!(

0 commit comments

Comments
 (0)