try to use lscolors gnu_legacy

Cargo.toml

@@ -288,7 +288,7 @@ indicatif = "0.17"
 itertools = "0.12.0"
 libc = "0.2.150"
 lscolors = { version = "0.16.0", default-features = false, features = [
-  "nu-ansi-term",
+  "nu-ansi-term", "gnu_legacy",
 ] }
 memchr = "2"
 memmap2 = "0.9"

src/uu/ls/Cargo.toml

@@ -22,7 +22,7 @@ number_prefix = { workspace = true }
 uutils_term_grid = { workspace = true }
 terminal_size = { workspace = true }
 glob = { workspace = true }
-lscolors = { workspace = true }
+lscolors = { workspace = true, features = ["gnu_legacy"] }
 uucore = { workspace = true, features = [
   "entries",
   "fs",

src/uu/ls/src/ls.rs

@@ -3097,7 +3097,7 @@ fn display_file_name(
 fn color_name(name: String, path: &Path, md: Option<&Metadata>, ls_colors: &LsColors) -> String {
     match ls_colors.style_for_path_with_metadata(path, md) {
         Some(style) => {
-            return style.to_nu_ansi_term_style().paint(name).to_string();
+            return style.to_nu_ansi_term_style().reset_before_style().paint(name).to_string();
         }
         None => name,
     }

tests/by-util/test_ls.rs

@@ -1174,7 +1174,9 @@ fn test_ls_long_symlink_color() {
         expected_target: &str,
     ) {
         // Names are always compared.
+        println!("name = {} / expected {}", to_ascii_representation(name), to_ascii_representation(expected_name));
         assert_eq!(&name, &expected_name);
+        println!("target = {} / expected {}", to_ascii_representation(target), to_ascii_representation(expected_target));
         assert_eq!(&target, &expected_target);
 
         // Colors are only compared when we have inferred what color we are looking for.
@@ -1962,6 +1964,17 @@ fn test_ls_recursive_1() {
         .stdout_is(out);
 }
 
+// Function to convert a string to its ASCII representation
+fn to_ascii_representation(input: &str) -> String {
+    input.chars().map(|c| {
+        if c.is_ascii_control() || !c.is_ascii() {
+            format!("\\x{:02x}", c as u32)
+        } else {
+            c.to_string()
+        }
+    }).collect::<String>()
+}
+
 #[test]
 fn test_ls_color() {
     let scene = TestScenario::new(util_name!());
@@ -1979,10 +1992,10 @@ fn test_ls_color() {
         .to_string();
     at.touch(nested_file);
     at.touch("test-color");
-
-    let a_with_colors = "\x1b[1;34ma\x1b[0m";
-    let z_with_colors = "\x1b[1;34mz\x1b[0m";
-    let nested_dir_with_colors = "\x1b[1;34mnested_dir\x1b[0m"; // spell-checker:disable-line
+                              
+    let a_with_colors = "\x1b[0m\x1b[01;34ma\x1b[0m\x0a";
+    let z_with_colors = "\x1b[0m\x1b[01;34mz\x1b[0m\x0a";
+    let nested_dir_with_colors = "\x1b[0m\x1b[01;34mnested_dir\x1b[0m\x0anested_file\x0a"; // spell-checker:disable-line
 
     // Color is disabled by default
     let result = scene.ucmd().succeeds();
@@ -1991,12 +2004,19 @@ fn test_ls_color() {
 
     // Color should be enabled
     for param in ["--color", "--col", "--color=always", "--col=always"] {
-        scene
+
+    println!(
+        "a_with_colors {} / z_with_colors {}",
+        to_ascii_representation(a_with_colors),
+        to_ascii_representation(z_with_colors)
+    );
+        let result = scene
             .ucmd()
             .arg(param)
-            .succeeds()
-            .stdout_contains(a_with_colors)
-            .stdout_contains(z_with_colors);
+            .succeeds();
+        println!("stdout = {} / ascii = {}", result.stdout_str(), to_ascii_representation(result.stdout_str()));
+        assert!(result.stdout_str().contains(a_with_colors));
+        assert!(result.stdout_str().contains(z_with_colors));
     }
 
     // Color should be disabled
@@ -2005,12 +2025,14 @@ fn test_ls_color() {
     assert!(!result.stdout_str().contains(z_with_colors));
 
     // Nested dir should be shown and colored
-    scene
+    let result = scene
         .ucmd()
         .arg("--color")
         .arg("a")
-        .succeeds()
-        .stdout_contains(nested_dir_with_colors);
+        .succeeds();
+    println!("stdout = {} / ascii = {}", result.stdout_str(), to_ascii_representation(result.stdout_str()));
+    assert!(result.stdout_str().contains(nested_dir_with_colors));
+
 
     // No output
     scene
@@ -2022,13 +2044,15 @@ fn test_ls_color() {
 
     // The colors must not mess up the grid layout
     at.touch("b");
-    scene
+    let result = scene
         .ucmd()
         .arg("--color")
         .arg("-w=15")
         .arg("-C")
-        .succeeds()
-        .stdout_only(format!("{a_with_colors}  test-color\nb  {z_with_colors}\n"));
+        .succeeds();
+    println!("stdout = {} / ascii = {}", result.stdout_str(), to_ascii_representation(result.stdout_str()));
+    println!("expected {}", to_ascii_representation("{a_with_colors}  test-color\nb  {z_with_colors}\n"));
+    assert!(result.stdout_str() == format!("{a_with_colors}  test-color\x0ab  {z_with_colors}\n"));
 }
 
 #[cfg(unix)]