ls: enable ANSI support on Windows for --hyperlink

uutils · Dec 6, 2023 · c947608 · c947608
1 parent de0a19d
commit c947608
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 0 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/src/uu/ls/Cargo.toml b/src/uu/ls/Cargo.toml
@@ -33,6 +33,9 @@ once_cell = { workspace = true }
 selinux = { workspace = true, optional = true }
 hostname = "0.3"
 
+[target.'cfg(windows)'.dependencies]
+nu-ansi-term = "0.49.0"
+
 [[bin]]
 name = "ls"
 path = "src/main.rs"

diff --git a/src/uu/ls/src/ls.rs b/src/uu/ls/src/ls.rs
@@ -2996,6 +2996,9 @@ fn display_file_name(
  let mut width = name.width();
 
  if config.hyperlink {
+ #[cfg(windows)]
+ let _enabled = nu_ansi_term::enable_ansi_support();
+
  let hostname = hostname::get().unwrap_or(OsString::from(""));
  let hostname = hostname.to_string_lossy();
 

diff --git a/tests/by-util/test_ls.rs b/tests/by-util/test_ls.rs
@@ -3840,6 +3840,7 @@ fn test_ls_hyperlink() {
  let path = at.root_dir_resolved();
 
  let result = scene.ucmd().arg("--hyperlink").succeeds();
+ println!("stdout: {}", result.stdout_str());
  assert!(result.stdout_str().contains("\x1b]8;;file://"));
  assert!(result
  .stdout_str()