cksum: transition to checksum_common

Author: RenjiSann

Cargo.lock

@@ -25,6 +25,7 @@ uucore = { workspace = true, features = [
   "sum",
   "hardware",
 ] }
+uu_checksum_common = { workspace = true }
 fluent = { workspace = true }
 
 [dev-dependencies]

src/uu/cksum/Cargo.toml

@@ -12,19 +12,3 @@ cksum-after-help = DIGEST determines the digest algorithm and default output for
   - sha3: (only available through cksum)
   - blake2b: (equivalent to b2sum)
   - sm3: (only available through cksum)
-
-# Help messages
-cksum-help-algorithm = select the digest type to use. See DIGEST below
-cksum-help-untagged = create a reversed style checksum, without digest type
-cksum-help-tag = create a BSD style checksum, undo --untagged (default)
-cksum-help-length = digest length in bits; must not exceed the max for the blake2 algorithm and must be a multiple of 8
-cksum-help-raw = emit a raw binary digest, not hexadecimal
-cksum-help-strict = exit non-zero for improperly formatted checksum lines
-cksum-help-check = read hashsums from the FILEs and check them
-cksum-help-base64 = emit a base64 digest, not hexadecimal
-cksum-help-warn = warn about improperly formatted checksum lines
-cksum-help-status = don't output anything, status code shows success
-cksum-help-quiet = don't print OK for each successfully verified file
-cksum-help-ignore-missing = don't fail or report status for missing files
-cksum-help-zero = end each output line with NUL, not newline, and disable file name escaping
-cksum-help-debug = print CPU hardware capability detection info used by cksum

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

@@ -12,19 +12,3 @@ cksum-after-help = DIGEST détermine l'algorithme de condensé et le format de s
   - sha3 : (disponible uniquement via cksum)
   - blake2b : (équivalent à b2sum)
   - sm3 : (disponible uniquement via cksum)
-
-# Messages d'aide
-cksum-help-algorithm = sélectionner le type de condensé à utiliser. Voir DIGEST ci-dessous
-cksum-help-untagged = créer une somme de contrôle de style inversé, sans type de condensé
-cksum-help-tag = créer une somme de contrôle de style BSD, annuler --untagged (par défaut)
-cksum-help-length = longueur du condensé en bits ; ne doit pas dépasser le maximum pour l'algorithme blake2 et doit être un multiple de 8
-cksum-help-raw = émettre un condensé binaire brut, pas hexadécimal
-cksum-help-strict = sortir avec un code non-zéro pour les lignes de somme de contrôle mal formatées
-cksum-help-check = lire les sommes de hachage des FICHIERs et les vérifier
-cksum-help-base64 = émettre un condensé base64, pas hexadécimal
-cksum-help-warn = avertir des lignes de somme de contrôle mal formatées
-cksum-help-status = ne rien afficher, le code de statut indique le succès
-cksum-help-quiet = ne pas afficher OK pour chaque fichier vérifié avec succès
-cksum-help-ignore-missing = ne pas échouer ou signaler le statut pour les fichiers manquants
-cksum-help-zero = terminer chaque ligne de sortie avec NUL, pas un saut de ligne, et désactiver l'échappement des noms de fichiers
-cksum-help-debug = afficher les informations de débogage sur la détection de la prise en charge matérielle du processeur

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

@@ -7,20 +7,16 @@
 
 use clap::builder::ValueParser;
 use clap::{Arg, ArgAction, Command};
-use std::ffi::OsString;
-use uucore::checksum::compute::{
-    ChecksumComputeOptions, OutputFormat, perform_checksum_computation,
-};
-use uucore::checksum::validate::{
-    ChecksumValidateOptions, ChecksumVerbose, perform_checksum_validation,
-};
+
+use uu_checksum_common::{checksum_main, options};
+
+use uucore::checksum::compute::OutputFormat;
 use uucore::checksum::{
-    AlgoKind, ChecksumError, SUPPORTED_ALGORITHMS, SizedAlgoKind, calculate_blake2b_length_str,
+    AlgoKind, ChecksumError, SUPPORTED_ALGORITHMS, calculate_blake2b_length_str,
     sanitize_sha2_sha3_length_str,
 };
 use uucore::error::UResult;
 use uucore::hardware::{HasHardwareFeatures as _, SimdPolicy};
-use uucore::line_ending::LineEnding;
 use uucore::{format_usage, show_error, translate};
 
 /// Print CPU hardware capability detection information to stderr
@@ -47,26 +43,6 @@ fn print_cpu_debug_info() {
     }
 }
 
-mod options {
-    pub const ALGORITHM: &str = "algorithm";
-    pub const FILE: &str = "file";
-    pub const UNTAGGED: &str = "untagged";
-    pub const TAG: &str = "tag";
-    pub const LENGTH: &str = "length";
-    pub const RAW: &str = "raw";
-    pub const BASE64: &str = "base64";
-    pub const CHECK: &str = "check";
-    pub const STRICT: &str = "strict";
-    pub const TEXT: &str = "text";
-    pub const BINARY: &str = "binary";
-    pub const STATUS: &str = "status";
-    pub const WARN: &str = "warn";
-    pub const IGNORE_MISSING: &str = "ignore-missing";
-    pub const QUIET: &str = "quiet";
-    pub const ZERO: &str = "zero";
-    pub const DEBUG: &str = "debug";
-}
-
 /// cksum has a bunch of legacy behavior. We handle this in this function to
 /// make sure they are self contained and "easier" to understand.
 ///
@@ -101,14 +77,6 @@ fn maybe_sanitize_length(
 pub fn uumain(args: impl uucore::Args) -> UResult<()> {
     let matches = uucore::clap_localization::handle_clap_result(uu_app(), args)?;
 
-    let check = matches.get_flag(options::CHECK);
-
-    let ignore_missing = matches.get_flag(options::IGNORE_MISSING);
-    let warn = matches.get_flag(options::WARN);
-    let quiet = matches.get_flag(options::QUIET);
-    let strict = matches.get_flag(options::STRICT);
-    let status = matches.get_flag(options::STATUS);
-
     let algo_cli = matches
         .get_one::<String>(options::ALGORITHM)
         .map(AlgoKind::from_cksum)
@@ -120,68 +88,22 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
 
     let length = maybe_sanitize_length(algo_cli, input_length)?;
 
-    // clap provides the default value -. So we unwrap() safety.
-    let files = matches
-        .get_many::<OsString>(options::FILE)
-        .unwrap()
-        .map(|s| s.as_os_str());
-
-    if check {
-        // cksum does not support '--check'ing legacy algorithms
-        if algo_cli.is_some_and(AlgoKind::is_legacy) {
-            return Err(ChecksumError::AlgorithmNotSupportedWithCheck.into());
-        }
-
-        let text_flag = matches.get_flag(options::TEXT);
-        let binary_flag = matches.get_flag(options::BINARY);
-        let tag = matches.get_flag(options::TAG);
-
-        if tag || binary_flag || text_flag {
-            return Err(ChecksumError::BinaryTextConflict.into());
-        }
-
-        // Execute the checksum validation based on the presence of files or the use of stdin
-
-        let verbose = ChecksumVerbose::new(status, quiet, warn);
-        let opts = ChecksumValidateOptions {
-            ignore_missing,
-            strict,
-            verbose,
-        };
-
-        return perform_checksum_validation(files, algo_cli, length, opts);
-    }
-
-    // Not --check
+    let output_format = OutputFormat::from_cksum(
+        algo_cli.unwrap_or(AlgoKind::Crc),
+        // Making TAG default at clap blocks --untagged
+        /* tag */
+        !matches.get_flag(options::UNTAGGED),
+        /* binary */ matches.get_flag(options::BINARY),
+        /* raw */ matches.get_flag(options::RAW),
+        /* base64 */ matches.get_flag(options::BASE64),
+    );
 
     // Print hardware debug info if requested
     if matches.get_flag(options::DEBUG) {
         print_cpu_debug_info();
     }
 
-    // Set the default algorithm to CRC when not '--check'ing.
-    let algo_kind = algo_cli.unwrap_or(AlgoKind::Crc);
-
-    let algo = SizedAlgoKind::from_unsized(algo_kind, length)?;
-    let line_ending = LineEnding::from_zero_flag(matches.get_flag(options::ZERO));
-
-    let opts = ChecksumComputeOptions {
-        algo_kind: algo,
-        output_format: OutputFormat::from_cksum(
-            algo_kind,
-            // Making TAG default at clap blocks --untagged
-            /* tag */
-            !matches.get_flag(options::UNTAGGED),
-            /* binary */ matches.get_flag(options::BINARY),
-            /* raw */ matches.get_flag(options::RAW),
-            /* base64 */ matches.get_flag(options::BASE64),
-        ),
-        line_ending,
-    };
-
-    perform_checksum_computation(opts, files)?;
-
-    Ok(())
+    checksum_main(algo_cli, length, matches, output_format)
 }
 
 pub fn uu_app() -> Command {