ichwh removed (#3349)

* ichwh removed

* removed unnecessary into() on PathBuf

Cargo.toml

@@ -82,9 +82,7 @@ rustyline-support = ["nu-cli/rustyline-support", "nu-command/rustyline-support"]
 term-support = ["nu-cli/term", "nu-command/term"]
 uuid-support = ["nu-cli/uuid_crate", "nu-command/uuid_crate"]
 which-support = [
-    "nu-cli/ichwh",
     "nu-cli/which",
-    "nu-command/ichwh",
     "nu-command/which",
     "nu-engine/which",
 ]

crates/nu-cli/Cargo.toml

@@ -57,7 +57,6 @@ getset = "0.1.1"
 glob = "0.3.0"
 htmlescape = "0.3.1"
 ical = "0.7.0"
-ichwh = { version = "0.3.4", optional = true }
 indexmap = { version = "1.6.1", features = ["serde-1"] }
 itertools = "0.10.0"
 lazy_static = "1.*"

crates/nu-command/Cargo.toml

@@ -53,7 +53,6 @@ getset = "0.1.1"
 glob = "0.3.0"
 htmlescape = "0.3.1"
 ical = "0.7.0"
-ichwh = { version = "0.3.4", optional = true }
 indexmap = { version = "1.6.1", features = ["serde-1"] }
 itertools = "0.10.0"
 lazy_static = "1.*"
@@ -97,7 +96,7 @@ trash = { version = "1.3.0", optional = true }
 unicode-segmentation = "1.7.1"
 url = "2.2.0"
 uuid_crate = { package = "uuid", version = "0.8.2", features = ["v4"], optional = true }
-which = { version = "4.0.2", optional = true }
+which = { version = "4.1.0", optional = true }
 zip = { version = "0.5.9", optional = true }
 
 [target.'cfg(unix)'.dependencies]

crates/nu-command/src/commands/version.rs

@@ -192,11 +192,6 @@ fn features_enabled() -> Vec<String> {
         names.push("which".to_string());
     }
 
-    #[cfg(feature = "ichwh")]
-    {
-        names.push("ichwh".to_string());
-    }
-
     #[cfg(feature = "zip")]
     {
         names.push("zip".to_string());

crates/nu-command/src/commands/which_.rs

@@ -135,32 +135,28 @@ macro_rules! entry_path {
     };
 }
 
-#[cfg(feature = "ichwh")]
+#[cfg(feature = "which")]
 fn get_first_entry_in_path(item: &str, tag: Tag) -> Option<Value> {
-    use futures::executor::block_on;
-
-    block_on(ichwh::which(item))
-        .unwrap_or(None)
-        .map(|path| entry_path!(item, path.into(), tag))
+    which::which(item)
+        .map(|path| entry_path!(item, path, tag))
+        .ok()
 }
 
-#[cfg(not(feature = "ichwh"))]
+#[cfg(not(feature = "which"))]
 fn get_first_entry_in_path(_: &str, _: Tag) -> Option<Value> {
     None
 }
 
-#[cfg(feature = "ichwh")]
+#[cfg(feature = "which")]
 fn get_all_entries_in_path(item: &str, tag: Tag) -> Vec<Value> {
-    use futures::executor::block_on;
-
-    block_on(ichwh::which_all(&item))
-        .unwrap_or_default()
-        .into_iter()
-        .map(|path| entry_path!(item, path.into(), tag.clone()))
+    which::which_all(&item)
+        .map(|iter| {
+            iter.map(|path| entry_path!(item, path, tag.clone()))
                 .collect()
+        })
+        .unwrap_or_default()
 }
-
-#[cfg(not(feature = "ichwh"))]
+#[cfg(not(feature = "which"))]
 fn get_all_entries_in_path(_: &str, _: Tag) -> Vec<Value> {
     vec![]
 }