numfmt: precision is ignored #11667

src/uu/numfmt/src/format.rs

@@ -486,6 +486,7 @@ fn transform_to(
     round_method: RoundMethod,
     precision: usize,
     unit_separator: &str,
+    is_specified: bool,
 ) -> Result<String> {
     let (i2, s) = consider_suffix(s, opts.to, round_method, precision)?;
     let i2 = i2 / (opts.to_unit as f64);
@@ -502,10 +503,16 @@ fn transform_to(
                 DisplayableSuffix(s, opts.to),
             )
         }
+        Some(s) if is_specified => {
+            format!("{i2:.0}{unit_separator}{}", DisplayableSuffix(s, opts.to))
+        }
         Some(s) if i2.abs() < 10.0 => {
+            // when there's a single digit before the dot.
             format!("{i2:.1}{unit_separator}{}", DisplayableSuffix(s, opts.to))
         }
-        Some(s) => format!("{i2:.0}{unit_separator}{}", DisplayableSuffix(s, opts.to)),
+        Some(s) => {
+            format!("{i2:.0}{unit_separator}{}", DisplayableSuffix(s, opts.to))
+        }
     })
 }
 
@@ -539,7 +546,7 @@ fn format_string(
         Some(suffix) => source.strip_suffix(suffix).unwrap_or(source),
         None => source,
     };
-
+    let mut is_specified = true;
     let precision = if let Some(p) = options.format.precision {
         p
     } else if options.transform.to == Unit::None
@@ -550,6 +557,7 @@ fn format_string(
     {
         parse_implicit_precision(source_without_suffix)
     } else {
+        is_specified = false;
         0
     };
 
@@ -559,6 +567,7 @@ fn format_string(
         options.round,
         precision,
         &options.unit_separator,
+        is_specified,
     )?;
 
     // bring back the suffix before applying padding
@@ -598,7 +607,6 @@ fn format_string(
         options.format.prefix, options.format.suffix
     ))
 }
-
 /// Encodes a byte slice as a string, representing non-UTF-8 bytes and non-printable ASCII
 /// bytes as octal escapes. Valid UTF-8 multi-byte characters pass through unchanged.
 /// Used to safely format invalid input in error messages.

tests/by-util/test_numfmt.rs

@@ -1452,7 +1452,6 @@ fn test_to_unit_prefix_selection_issue_11666() {
 // `--format='%.0f'` with `--to=<scale>` still prints one fractional digit;
 // the precision specifier `.0` is ignored.
 #[test]
-#[ignore = "GNU compat: see uutils/coreutils#11667"]
 fn test_format_precision_zero_with_to_scale_issue_11667() {
     new_ucmd!()
         .args(&["--to=iec", "--format=%.0f", "5183776"])