hashsum, cksum: Move --ckeck confliction to clap

Author: oech3

src/uu/cksum/src/cksum.rs

@@ -134,14 +134,6 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
             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);
@@ -251,6 +243,9 @@ pub fn uu_app() -> Command {
                 .short('c')
                 .long(options::CHECK)
                 .help(translate!("cksum-help-check"))
+                .conflicts_with(options::TAG)
+                .conflicts_with(options::BINARY)
+                .conflicts_with(options::TEXT)
                 .action(ArgAction::SetTrue),
         )
         .arg(

src/uu/hashsum/src/hashsum.rs

@@ -171,18 +171,7 @@ pub fn uumain(mut args: impl uucore::Args) -> UResult<()> {
     );
 
     if check {
-        // on Windows, allow --binary/--text to be used with --check
-        // and keep the behavior of defaulting to binary
-        #[cfg(not(windows))]
-        {
-            let text_flag = matches.get_flag("text");
-            let binary_flag = matches.get_flag("binary");
-
-            if binary_flag || text_flag {
-                return Err(ChecksumError::BinaryTextConflict.into());
-            }
-        }
-
+        // No reason to allow --check with --binary/--text on Cygwin. It want to be same with Linux and --text was broken for a long time.
         let verbose = ChecksumVerbose::new(status, quiet, warn);
 
         let opts = ChecksumValidateOptions {
@@ -261,7 +250,9 @@ pub fn uu_app_common() -> Command {
                 .long("check")
                 .help(translate!("hashsum-help-check"))
                 .action(ArgAction::SetTrue)
-                .conflicts_with("tag"),
+                .conflicts_with(options::BINARY)
+                .conflicts_with(options::TEXT)
+                .conflicts_with(options::TAG),
         )
         .arg(
             Arg::new(options::TAG)

tests/by-util/test_cksum.rs

@@ -1216,7 +1216,7 @@ fn test_conflicting_options() {
         .fails_with_code(1)
         .no_stdout()
         .stderr_contains(
-            "cksum: the --binary and --text options are meaningless when verifying checksums",
+           "cannot be used with", //clap generated error
         );
 
     scene
@@ -1228,7 +1228,7 @@ fn test_conflicting_options() {
         .fails_with_code(1)
         .no_stdout()
         .stderr_contains(
-            "cksum: the --binary and --text options are meaningless when verifying checksums",
+            "cannot be used with", //clap generated error
         );
 }