sha512sum: introduce standalone binary

RenjiSann · RenjiSann · commit fc4ea59b3ac8 · 2026-01-11T04:17:33.000+01:00
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -92,6 +92,7 @@ feat_common_core = [
   "sha224sum",
   "sha256sum",
   "sha384sum",
+  "sha512sum",
   "comm",
   "cp",
   "csplit",
@@ -446,6 +447,7 @@ sha1sum = { optional = true, version = "0.5.0", package = "uu_sha1sum", path = "
 sha224sum = { optional = true, version = "0.5.0", package = "uu_sha224sum", path = "src/uu/sha224sum" }
 sha256sum = { optional = true, version = "0.5.0", package = "uu_sha256sum", path = "src/uu/sha256sum" }
 sha384sum = { optional = true, version = "0.5.0", package = "uu_sha384sum", path = "src/uu/sha384sum" }
+sha512sum = { optional = true, version = "0.5.0", package = "uu_sha512sum", path = "src/uu/sha512sum" }
 comm = { optional = true, version = "0.5.0", package = "uu_comm", path = "src/uu/comm" }
 cp = { optional = true, version = "0.5.0", package = "uu_cp", path = "src/uu/cp" }
 csplit = { optional = true, version = "0.5.0", package = "uu_csplit", path = "src/uu/csplit" }
diff --git a/GNUmakefile b/GNUmakefile
@@ -90,6 +90,7 @@ PROGS       := \
 	sha224sum \
 	sha256sum \
 	sha384sum \
+	sha512sum \
 	comm \
 	cp \
 	csplit \
@@ -190,9 +191,6 @@ SELINUX_PROGS := \
 	chcon \
 	runcon
 
-HASHSUM_PROGS := \
-	sha512sum
-
 $(info Detected OS = $(OS))
 
 ifeq (,$(findstring MINGW,$(OS)))
@@ -229,6 +227,7 @@ TEST_PROGS  := \
 	sha224sum \
 	sha256sum \
 	sha384sum \
+	sha512sum \
 	comm \
 	cp \
 	csplit \
diff --git a/build.rs b/build.rs
@@ -77,9 +77,6 @@ pub fn main() {
             }
             "hashsum" => {
                 phf_map.entry(krate, format!("({krate}::uumain, {krate}::uu_app_custom)"));
-
-                let map_value = format!("({krate}::uumain, {krate}::uu_app_common)");
-                phf_map.entry("sha512sum", map_value.clone());
             }
             _ => {
                 phf_map.entry(krate, map_value.clone());
diff --git a/src/common/validation.rs b/src/common/validation.rs
@@ -50,9 +50,6 @@ fn get_canonical_util_name(util_name: &str) -> &str {
         // uu_test aliases - '[' is an alias for test
         "[" => "test",
 
-        // hashsum aliases - all these hash commands are aliases for hashsum
-        "sha512sum" => "hashsum",
-
         "dir" => "ls", // dir is an alias for ls
 
         // Default case - return the util name as is
diff --git a/src/uu/sha512sum/Cargo.toml b/src/uu/sha512sum/Cargo.toml
@@ -0,0 +1,38 @@
+[package]
+name = "uu_sha512sum"
+description = "sha512sum ~ (uutils) Print or check the SHA512 checksums"
+repository = "https://github.com/uutils/coreutils/tree/main/src/uu/sha512sum"
+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/sha512sum.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 = "sha512sum"
+path = "src/main.rs"
diff --git a/src/uu/sha512sum/LICENSE b/src/uu/sha512sum/LICENSE
@@ -0,0 +1 @@
+../../../LICENSE
diff --git a/src/uu/sha512sum/locales/en-US.ftl b/src/uu/sha512sum/locales/en-US.ftl
@@ -0,0 +1,2 @@
+sha512sum-about = Print or check the SHA512 checksums
+sha512sum-usage = sha512sum [OPTIONS] [FILE]...
diff --git a/src/uu/sha512sum/locales/fr-FR.ftl b/src/uu/sha512sum/locales/fr-FR.ftl
@@ -0,0 +1,2 @@
+sha512sum-about = Afficher le SHA512 et la taille de chaque fichier
+sha512sum-usage = sha512sum [OPTION]... [FICHIER]...
diff --git a/src/uu/sha512sum/src/main.rs b/src/uu/sha512sum/src/main.rs
@@ -0,0 +1 @@
+uucore::bin!(uu_sha512sum);
diff --git a/src/uu/sha512sum/src/sha512sum.rs b/src/uu/sha512sum/src/sha512sum.rs
@@ -0,0 +1,24 @@
+// 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.
+
+// spell-checker:ignore (ToDO) algo
+
+use clap::Command;
+
+use uu_checksum_common::{standalone_checksum_app, standalone_main};
+
+use uucore::checksum::AlgoKind;
+use uucore::error::UResult;
+use uucore::translate;
+
+#[uucore::main]
+pub fn uumain(args: impl uucore::Args) -> UResult<()> {
+    standalone_main(AlgoKind::Sha512, uu_app(), args)
+}
+
+#[inline]
+pub fn uu_app() -> Command {
+    standalone_checksum_app(translate!("sha512sum-about"), translate!("sha512sum-usage"))
+}
diff --git a/src/uucore/src/lib/lib.rs b/src/uucore/src/lib/lib.rs
@@ -172,9 +172,6 @@ pub fn get_canonical_util_name(util_name: &str) -> &str {
         // uu_test aliases - '[' is an alias for test
         "[" => "test",
 
-        // hashsum aliases - all these hash commands are aliases for hashsum
-        "sha512sum" => "hashsum",
-
         "dir" => "ls", // dir is an alias for ls
 
         // Default case - return the util name as is
diff --git a/src/uucore/src/lib/mods/locale.rs b/src/uucore/src/lib/mods/locale.rs
@@ -165,6 +165,7 @@ fn create_bundle(
         "sha224sum",
         "sha256sum",
         "sha384sum",
+        "sha512sum",
     ]
     .contains(&util_name)
     {
diff --git a/tests/by-util/test_hashsum.rs b/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! {sha512, sha512, 512}
 test_digest_with_len! {sha3_224, sha3, 224}
 test_digest_with_len! {sha3_256, sha3, 256}
 test_digest_with_len! {sha3_384, sha3, 384}
diff --git a/tests/by-util/test_sha512sum.rs b/tests/by-util/test_sha512sum.rs
@@ -0,0 +1,128 @@
+// 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! {sha512}
+
+#[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: sha512sum")
+        .stdout_does_not_contain("Usage: hashsum");
+}
diff --git a/tests/fixtures/sha512sum/input.txt b/tests/fixtures/sha512sum/input.txt
@@ -0,0 +1 @@
+hello, world
diff --git a/tests/fixtures/sha512sum/sha512.checkfile b/tests/fixtures/sha512sum/sha512.checkfile
@@ -0,0 +1 @@
+8710339dcb6814d0d9d2290ef422285c9322b7163951f9a0ca8f883d3305286f44139aa374848e4174f5aada663027e4548637b6d19894aec4fb6c46a139fbf9  input.txt
diff --git a/tests/fixtures/sha512sum/sha512.expected b/tests/fixtures/sha512sum/sha512.expected
@@ -0,0 +1 @@
+8710339dcb6814d0d9d2290ef422285c9322b7163951f9a0ca8f883d3305286f44139aa374848e4174f5aada663027e4548637b6d19894aec4fb6c46a139fbf9
diff --git a/tests/tests.rs b/tests/tests.rs
@@ -92,6 +92,10 @@ mod test_sha256sum;
 #[path = "by-util/test_sha384sum.rs"]
 mod test_sha384sum;
 
+#[cfg(feature = "sha512sum")]
+#[path = "by-util/test_sha512sum.rs"]
+mod test_sha512sum;
+
 #[cfg(feature = "cp")]
 #[path = "by-util/test_cp.rs"]
 mod test_cp;

Original file line number	Diff line number	Diff line change
`@@ -77,9 +77,6 @@ pub fn main() {`
`77`	`77`	`}`
`78`	`78`	`"hashsum" => {`
`79`	`79`	`phf_map.entry(krate, format!("({krate}::uumain, {krate}::uu_app_custom)"));`
`80`		`-`
`81`		`- let map_value = format!("({krate}::uumain, {krate}::uu_app_common)");`
`82`		`- phf_map.entry("sha512sum", map_value.clone());`
`83`	`80`	`}`
`84`	`81`	`_ => {`
`85`	`82`	`phf_map.entry(krate, map_value.clone());`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+sha512sum-about = Print or check the SHA512 checksums`
	`2`	`+sha512sum-usage = sha512sum [OPTIONS] [FILE]...`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+sha512sum-about = Afficher le SHA512 et la taille de chaque fichier`
	`2`	`+sha512sum-usage = sha512sum [OPTION]... [FICHIER]...`