sha384sum: introduce standalone binary

Cargo.lock

@@ -630,6 +630,7 @@ dependencies = [
  "uu_sha1sum",
  "uu_sha224sum",
  "uu_sha256sum",
+ "uu_sha384sum",
  "uu_shred",
  "uu_shuf",
  "uu_sleep",
@@ -4023,6 +4024,18 @@ dependencies = [
  "uucore",
 ]
 
+[[package]]
+name = "uu_sha384sum"
+version = "0.6.0"
+dependencies = [
+ "clap",
+ "codspeed-divan-compat",
+ "fluent",
+ "tempfile",
+ "uu_checksum_common",
+ "uucore",
+]
+
 [[package]]
 name = "uu_shred"
 version = "0.6.0"

Cargo.toml

@@ -91,6 +91,7 @@ feat_common_core = [
   "sha1sum",
   "sha224sum",
   "sha256sum",
+  "sha384sum",
   "comm",
   "cp",
   "csplit",
@@ -447,6 +448,7 @@ md5sum = { optional = true, version = "0.6.0", package = "uu_md5sum", path = "sr
 sha1sum = { optional = true, version = "0.6.0", package = "uu_sha1sum", path = "src/uu/sha1sum" }
 sha224sum = { optional = true, version = "0.6.0", package = "uu_sha224sum", path = "src/uu/sha224sum" }
 sha256sum = { optional = true, version = "0.6.0", package = "uu_sha256sum", path = "src/uu/sha256sum" }
+sha384sum = { optional = true, version = "0.6.0", package = "uu_sha384sum", path = "src/uu/sha384sum" }
 comm = { optional = true, version = "0.6.0", package = "uu_comm", path = "src/uu/comm" }
 cp = { optional = true, version = "0.6.0", package = "uu_cp", path = "src/uu/cp" }
 csplit = { optional = true, version = "0.6.0", package = "uu_csplit", path = "src/uu/csplit" }

GNUmakefile

@@ -96,7 +96,6 @@ SELINUX_PROGS := \
 	runcon
 
 HASHSUM_PROGS := \
-	sha384sum \
 	sha512sum
 
 $(info Detected OS = $(OS))

build.rs

@@ -91,7 +91,6 @@ pub fn main() {
                 phf_map.entry(krate, format!("({krate}::uumain, {krate}::uu_app_custom)"));
 
                 let map_value = format!("({krate}::uumain, {krate}::uu_app_common)");
-                phf_map.entry("sha384sum", map_value.clone());
                 phf_map.entry("sha512sum", map_value.clone());
             }
             _ => {

src/common/validation.rs

@@ -51,7 +51,7 @@ fn get_canonical_util_name(util_name: &str) -> &str {
         "[" => "test",
 
         // hashsum aliases - all these hash commands are aliases for hashsum
-        "sha384sum" | "sha512sum" => "hashsum",
+        "sha512sum" => "hashsum",
 
         "dir" => "ls", // dir is an alias for ls
 

src/uu/sha384sum/Cargo.toml

@@ -0,0 +1,38 @@
+[package]
+name = "uu_sha384sum"
+description = "sha384sum ~ (uutils) Print or check the SHA384 checksums"
+repository = "https://github.com/uutils/coreutils/tree/main/src/uu/sha384sum"
+version.workspace = true
+authors.workspace = true
+license.workspace = true
+homepage.workspace = true
+keywords.workspace = true
+categories.workspace = true
+edition.workspace = true
+readme.workspace = true
+
+[lints]
+workspace = true
+
+[lib]
+path = "src/sha384sum.rs"
+
+[dependencies]
+clap = { workspace = true }
+uu_checksum_common = { workspace = true }
+uucore = { workspace = true, features = [
+  "checksum",
+  "encoding",
+  "sum",
+  "hardware",
+] }
+fluent = { workspace = true }
+
+[dev-dependencies]
+divan = { workspace = true }
+tempfile = { workspace = true }
+uucore = { workspace = true, features = ["benchmark"] }
+
+[[bin]]
+name = "sha384sum"
+path = "src/main.rs"

src/uu/sha384sum/LICENSE

@@ -0,0 +1 @@
+../../../LICENSE

src/uu/sha384sum/locales/en-US.ftl

@@ -0,0 +1,2 @@
+sha384sum-about = Print or check the SHA384 checksums
+sha384sum-usage = sha384sum [OPTIONS] [FILE]...

src/uu/sha384sum/locales/fr-FR.ftl

@@ -0,0 +1,2 @@
+sha1sum-about = Afficher le SHA384 et la taille de chaque fichier
+sha1sum-usage = sha384sum [OPTION]... [FICHIER]...

src/uu/sha384sum/src/main.rs

@@ -0,0 +1 @@
+uucore::bin!(uu_sha384sum);

src/uu/sha384sum/src/sha384sum.rs

@@ -0,0 +1 @@
+uu_checksum_common::declare_standalone!("sha384sum", uucore::checksum::AlgoKind::Sha384);

src/uucore/src/lib/lib.rs

@@ -173,7 +173,7 @@ pub fn get_canonical_util_name(util_name: &str) -> &str {
         "[" => "test",
 
         // hashsum aliases - all these hash commands are aliases for hashsum
-        "sha384sum" | "sha512sum" => "hashsum",
+        "sha512sum" => "hashsum",
 
         "dir" => "ls", // dir is an alias for ls
 

src/uucore/src/lib/mods/locale.rs

@@ -164,6 +164,7 @@ fn create_bundle(
         "sha1sum",
         "sha224sum",
         "sha256sum",
+        "sha384sum",
     ]
     .contains(&util_name)
     {

tests/by-util/test_hashsum.rs

@@ -205,7 +205,6 @@ test_digest! {b3sum, b3sum}
 test_digest! {shake128, shake128}
 test_digest! {shake256, shake256}
 
-test_digest_with_len! {sha384, sha384, 384}
 test_digest_with_len! {sha512, sha512, 512}
 test_digest_with_len! {sha3_224, sha3, 224}
 test_digest_with_len! {sha3_256, sha3, 256}

tests/by-util/test_sha384sum.rs

@@ -0,0 +1,122 @@
+// This file is part of the uutils coreutils package.
+//
+// For the full copyright and license information, please view the LICENSE
+// file that was distributed with this source code.
+
+use uutests::new_ucmd;
+// spell-checker:ignore checkfile, testf, ntestf
+macro_rules! get_hash(
+    ($str:expr) => (
+        $str.split(' ').collect::<Vec<&str>>()[0]
+    );
+);
+
+macro_rules! test_digest {
+    ($id:ident) => {
+        mod $id {
+            use uutests::util::*;
+            use uutests::util_name;
+            static EXPECTED_FILE: &'static str = concat!(stringify!($id), ".expected");
+            static CHECK_FILE: &'static str = concat!(stringify!($id), ".checkfile");
+            static INPUT_FILE: &'static str = "input.txt";
+
+            #[test]
+            fn test_single_file() {
+                let ts = TestScenario::new(util_name!());
+                assert_eq!(
+                    ts.fixtures.read(EXPECTED_FILE),
+                    get_hash!(
+                        ts.ucmd()
+                            .arg(INPUT_FILE)
+                            .succeeds()
+                            .no_stderr()
+                            .stdout_str()
+                    )
+                );
+            }
+
+            #[test]
+            fn test_stdin() {
+                let ts = TestScenario::new(util_name!());
+                assert_eq!(
+                    ts.fixtures.read(EXPECTED_FILE),
+                    get_hash!(
+                        ts.ucmd()
+                            .pipe_in_fixture(INPUT_FILE)
+                            .succeeds()
+                            .no_stderr()
+                            .stdout_str()
+                    )
+                );
+            }
+
+            #[test]
+            fn test_check() {
+                let ts = TestScenario::new(util_name!());
+                println!("File content='{}'", ts.fixtures.read(INPUT_FILE));
+                println!("Check file='{}'", ts.fixtures.read(CHECK_FILE));
+
+                ts.ucmd()
+                    .args(&["--check", CHECK_FILE])
+                    .succeeds()
+                    .no_stderr()
+                    .stdout_is("input.txt: OK\n");
+            }
+
+            #[test]
+            fn test_zero() {
+                let ts = TestScenario::new(util_name!());
+                assert_eq!(
+                    ts.fixtures.read(EXPECTED_FILE),
+                    get_hash!(
+                        ts.ucmd()
+                            .arg("--zero")
+                            .arg(INPUT_FILE)
+                            .succeeds()
+                            .no_stderr()
+                            .stdout_str()
+                    )
+                );
+            }
+
+            #[test]
+            fn test_missing_file() {
+                let ts = TestScenario::new(util_name!());
+                let at = &ts.fixtures;
+
+                at.write("a", "file1\n");
+                at.write("c", "file3\n");
+
+                ts.ucmd()
+                    .args(&["a", "b", "c"])
+                    .fails()
+                    .stdout_contains("a\n")
+                    .stdout_contains("c\n")
+                    .stderr_contains("b: No such file or directory");
+            }
+        }
+    };
+}
+
+test_digest! {sha384}
+
+#[test]
+fn test_invalid_arg() {
+    new_ucmd!().arg("--definitely-invalid").fails_with_code(1);
+}
+
+#[test]
+fn test_conflicting_arg() {
+    new_ucmd!().arg("--tag").arg("--check").fails_with_code(1);
+    new_ucmd!().arg("--tag").arg("--text").fails_with_code(1);
+}
+
+#[test]
+fn test_help_shows_correct_utility_name() {
+    // Test that help output shows the actual utility name instead of "hashsum"
+    new_ucmd!()
+        .arg("--help")
+        .succeeds()
+        .stdout_contains("Usage: sha384sum")
+        .stdout_does_not_contain("Usage: hashsum");
+}

tests/fixtures/sha384sum/input.txt

@@ -0,0 +1 @@
+hello, world

tests/tests.rs

@@ -88,6 +88,10 @@ mod test_sha224sum;
 #[path = "by-util/test_sha256sum.rs"]
 mod test_sha256sum;
 
+#[cfg(feature = "sha384sum")]
+#[path = "by-util/test_sha384sum.rs"]
+mod test_sha384sum;
+
 #[cfg(feature = "cp")]
 #[path = "by-util/test_cp.rs"]
 mod test_cp;