review feedback

Author: rukai

src/lib.rs

@@ -141,15 +141,9 @@ fn do_locate_java_home() -> Result<String> {
         .unwrap()
         .trim();
 
-    if java_exec_path.is_empty() {
-        return Err(JavaLocatorError::new(
-            "Java is not installed or not in the system PATH".into(),
-        ));
-    }
-
+    java_exec_path_validation(java_exec_path)?;
     let mut home_path = follow_symlinks(java_exec_path);
 
-    // Here we should have found ourselves in a directory like /usr/lib/jvm/java-8-oracle/jre/bin/java
     home_path.pop();
     home_path.pop();
 
@@ -171,12 +165,7 @@ fn do_locate_java_home() -> Result<String> {
 
     let java_exec_path = std::str::from_utf8(&output.stdout)?.trim();
 
-    if java_exec_path.is_empty() {
-        return Err(JavaLocatorError::new(
-            "Java is not installed or not in the system PATH".into(),
-        ));
-    }
-
+    java_exec_path_validation(java_exec_path)?;
     let home_path = follow_symlinks(java_exec_path);
 
     home_path
@@ -193,12 +182,7 @@ fn do_locate_java_home() -> Result<String> {
         .map_err(|e| JavaLocatorError::new(format!("Failed to run command `which` ({e})")))?;
     let java_exec_path = std::str::from_utf8(&output.stdout)?.trim();
 
-    if java_exec_path.is_empty() {
-        return Err(JavaLocatorError::new(
-            "Java is not installed or not in the system PATH".into(),
-        ));
-    }
-
+    java_exec_path_validation(java_exec_path)?;
     let mut home_path = follow_symlinks(java_exec_path);
 
     // Here we should have found ourselves in a directory like /usr/lib/jvm/java-8-oracle/jre/bin/java
@@ -211,7 +195,21 @@ fn do_locate_java_home() -> Result<String> {
         .map_err(|path| JavaLocatorError::new(format!("Java path {path:?} is invalid utf8")))
 }
 
-// Its not clear to me which systems need this so for now its run on all systems.
+fn java_exec_path_validation(path: &str) -> Result<()> {
+    if path.is_empty() {
+        return Err(JavaLocatorError::new(
+            "Java is not installed or not in the system PATH".into(),
+        ));
+    }
+
+    let paths_found = path.lines().count();
+    if paths_found > 1 {
+        eprintln!("WARNING: java_locator found {paths_found} possible java locations. Using the first one. To silence this warning set JAVA_HOME env var.")
+    }
+
+    Ok(())
+}
+
 fn follow_symlinks(path: &str) -> PathBuf {
     let mut test_path = PathBuf::from(path);
     while let Ok(path) = test_path.read_link() {