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

@@ -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 {