From ea55b19ceef38a31465920d42afe82980a320a5c Mon Sep 17 00:00:00 2001
From: Daniel Hofstetter <daniel.hofstetter@42dh.com>
Date: Wed, 3 Aug 2022 16:14:05 +0200
Subject: [PATCH] pr: use clap to handle help & version

---
 src/uu/pr/src/pr.rs      | 33 +++------------------------------
 tests/by-util/test_pr.rs | 10 ++++++++++
 2 files changed, 13 insertions(+), 30 deletions(-)

diff --git a/src/uu/pr/src/pr.rs b/src/uu/pr/src/pr.rs
index c9364a3b22f..1cc6123a0a2 100644
--- a/src/uu/pr/src/pr.rs
+++ b/src/uu/pr/src/pr.rs
@@ -9,7 +9,7 @@
 #[macro_use]
 extern crate quick_error;
 
-use clap::{AppSettings, Arg, ArgMatches, Command};
+use clap::{crate_version, Arg, ArgMatches, Command};
 use itertools::Itertools;
 use quick_error::ResultExt;
 use regex::Regex;
@@ -22,9 +22,8 @@ use time::macros::format_description;
 use time::OffsetDateTime;
 
 use uucore::display::Quotable;
-use uucore::error::{set_exit_code, UResult};
+use uucore::error::UResult;
 
-const VERSION: &str = env!("CARGO_PKG_VERSION");
 const ABOUT: &str =
     "Write content of given file or standard input to standard output with pagination filter";
 const AFTER_HELP: &str =
@@ -84,8 +83,6 @@ mod options {
     pub const MERGE: &str = "merge";
     pub const INDENT: &str = "indent";
     pub const JOIN_LINES: &str = "join-lines";
-    pub const HELP: &str = "help";
-    pub const VERSION: &str = "version";
     pub const FILES: &str = "files";
 }
 
@@ -196,13 +193,11 @@ quick_error! {
 
 pub fn uu_app<'a>() -> Command<'a> {
     Command::new(uucore::util_name())
-        .version(VERSION)
+        .version(crate_version!())
         .about(ABOUT)
         .after_help(AFTER_HELP)
         .infer_long_args(true)
         .args_override_self(true)
-        .setting(AppSettings::NoAutoHelp)
-        .setting(AppSettings::NoAutoVersion)
         .arg(
             Arg::new(options::PAGES)
                 .long(options::PAGES)
@@ -359,17 +354,6 @@ pub fn uu_app<'a>() -> Command<'a> {
                 .help("merge full lines, turns off -W line truncation, no column
                 alignment, --sep-string[=STRING] sets separators")
         )
-        .arg(
-            Arg::new(options::HELP)
-                .long(options::HELP)
-                .help("Show this help message")
-        )
-        .arg(
-            Arg::new(options::VERSION)
-                .short('V')
-                .long(options::VERSION)
-                .help("Show version information")
-        )
         .arg(
             Arg::new(options::FILES)
                 .multiple_occurrences(true)
@@ -389,21 +373,10 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
         Ok(m) => m,
         Err(e) => {
             e.print()?;
-            set_exit_code(1);
             return Ok(());
         }
     };
 
-    if matches.contains_id(options::VERSION) {
-        println!("{}", command.render_long_version());
-        return Ok(());
-    }
-
-    if matches.contains_id(options::HELP) {
-        command.print_help()?;
-        return Ok(());
-    }
-
     let mut files = matches
         .get_many::<String>(options::FILES)
         .map(|v| v.map(|s| s.as_str()).collect::<Vec<_>>())
diff --git a/tests/by-util/test_pr.rs b/tests/by-util/test_pr.rs
index ec6df601f4a..20abd1a24fc 100644
--- a/tests/by-util/test_pr.rs
+++ b/tests/by-util/test_pr.rs
@@ -469,3 +469,13 @@ fn test_value_for_number_lines() {
     // Therefore, pr tries to access the file, which does not exist.
     new_ucmd!().args(&["-n", "foo5.txt", "test.log"]).fails();
 }
+
+#[test]
+fn test_help() {
+    new_ucmd!().arg("--help").succeeds();
+}
+
+#[test]
+fn test_version() {
+    new_ucmd!().arg("--version").succeeds();
+}