Merge pull request #8542 from sylvestre/clap-loca-improv-2

clap/locale: fix the colors for all programs (Closes: #8501)

Author: cakebaker

.vscode/cSpell.json

@@ -29,7 +29,8 @@
     "**/*.svg",
     "src/uu/*/locales/*.ftl",
     "src/uucore/locales/*.ftl",
-    ".devcontainer/**"
+    ".devcontainer/**",
+    "util/gnu-patches/**",
   ],
 
   "enableGlobDot": true,

.vscode/cspell.dictionaries/workspace.wordlist.txt

@@ -349,3 +349,12 @@ getcwd
 # * other
 weblate
 algs
+
+# translation tests
+CLICOLOR
+erreur
+Utilisation
+merror
+merreur
+verbo
+inattendu

Cargo.toml

@@ -301,7 +301,7 @@ chrono = { version = "0.4.41", default-features = false, features = [
   "alloc",
   "clock",
 ] }
-clap = { version = "4.5", features = ["wrap_help", "cargo"] }
+clap = { version = "4.5", features = ["wrap_help", "cargo", "color"] }
 clap_complete = "4.4"
 clap_mangen = "0.2"
 compare = "0.1.0"

src/uu/arch/src/arch.rs

@@ -6,13 +6,12 @@
 use platform_info::*;
 
 use clap::Command;
-use uucore::LocalizedCommand;
 use uucore::error::{UResult, USimpleError};
 use uucore::translate;
 
 #[uucore::main]
 pub fn uumain(args: impl uucore::Args) -> UResult<()> {
-    uu_app().get_matches_from_localized(args);
+    uucore::clap_localization::handle_clap_result(uu_app(), args)?;
 
     let uts =
         PlatformInfo::new().map_err(|_e| USimpleError::new(1, translate!("cannot-get-system")))?;

src/uu/base32/src/base_common.rs

@@ -10,7 +10,6 @@ use std::ffi::OsString;
 use std::fs::File;
 use std::io::{self, ErrorKind, Read, Seek, SeekFrom};
 use std::path::{Path, PathBuf};
-use uucore::LocalizedCommand;
 use uucore::display::Quotable;
 use uucore::encoding::{
     BASE2LSBF, BASE2MSBF, EncodingWrapper, Format, SupportsFastDecodeAndEncode, Z85Wrapper,
@@ -101,17 +100,17 @@ pub fn parse_base_cmd_args(
     usage: &str,
 ) -> UResult<Config> {
     let command = base_app(about, usage);
-    let matches = command.get_matches_from_localized(args);
+    let matches = uucore::clap_localization::handle_clap_result(command, args)?;
     Config::from(&matches)
 }
 
 pub fn base_app(about: &'static str, usage: &str) -> Command {
-    Command::new(uucore::util_name())
+    let cmd = Command::new(uucore::util_name())
         .version(uucore::crate_version!())
-        .help_template(uucore::localized_help_template(uucore::util_name()))
         .about(about)
         .override_usage(format_usage(usage))
-        .infer_long_args(true)
+        .infer_long_args(true);
+    uucore::clap_localization::configure_localized_command(cmd)
         // Format arguments.
         .arg(
             Arg::new(options::DECODE)

src/uu/basename/src/basename.rs

@@ -15,7 +15,6 @@ use uucore::error::{UResult, UUsageError};
 use uucore::format_usage;
 use uucore::line_ending::LineEnding;
 
-use uucore::LocalizedCommand;
 use uucore::translate;
 
 pub mod options {
@@ -30,7 +29,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
     //
     // Argument parsing
     //
-    let matches = uu_app().get_matches_from_localized(args);
+    let matches = uucore::clap_localization::handle_clap_result(uu_app(), args)?;
 
     let line_ending = LineEnding::from_zero_flag(matches.get_flag(options::ZERO));
 

src/uu/basenc/src/basenc.rs

@@ -7,7 +7,6 @@
 
 use clap::{Arg, ArgAction, Command};
 use uu_base32::base_common::{self, BASE_CMD_PARSE_ERROR, Config};
-use uucore::error::UClapError;
 use uucore::translate;
 use uucore::{
     encoding::Format,
@@ -64,9 +63,7 @@ pub fn uu_app() -> Command {
 }
 
 fn parse_cmd_args(args: impl uucore::Args) -> UResult<(Config, Format)> {
-    let matches = uu_app()
-        .try_get_matches_from(args.collect_lossy())
-        .with_exit_code(1)?;
+    let matches = uucore::clap_localization::handle_clap_result(uu_app(), args.collect_lossy())?;
 
     let encodings = get_encodings();
     let format = encodings

src/uu/cat/src/cat.rs

@@ -23,7 +23,6 @@ use std::os::unix::fs::FileTypeExt;
 #[cfg(unix)]
 use std::os::unix::net::UnixStream;
 use thiserror::Error;
-use uucore::LocalizedCommand;
 use uucore::display::Quotable;
 use uucore::error::UResult;
 #[cfg(not(target_os = "windows"))]
@@ -232,7 +231,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
         libc::signal(libc::SIGPIPE, libc::SIG_DFL);
     }
 
-    let matches = uu_app().get_matches_from_localized(args);
+    let matches = uucore::clap_localization::handle_clap_result(uu_app(), args)?;
 
     let number_mode = if matches.get_flag(options::NUMBER_NONBLANK) {
         NumberingMode::NonEmpty

src/uu/chcon/src/chcon.rs

@@ -7,12 +7,11 @@
 #![allow(clippy::upper_case_acronyms)]
 
 use clap::builder::ValueParser;
-use uucore::LocalizedCommand;
 use uucore::error::{UResult, USimpleError, UUsageError};
 use uucore::translate;
 use uucore::{display::Quotable, format_usage, show_error, show_warning};
 
-use clap::{Arg, ArgAction, Command};
+use clap::{Arg, ArgAction, ArgMatches, Command};
 use selinux::{OpaqueSecurityContext, SecurityContext};
 
 use std::borrow::Cow;
@@ -58,9 +57,9 @@ pub mod options {
 
 #[uucore::main]
 pub fn uumain(args: impl uucore::Args) -> UResult<()> {
-    let config = uu_app();
+    let matches = uucore::clap_localization::handle_clap_result(uu_app(), args)?;
 
-    let options = match parse_command_line(config, args) {
+    let options = match parse_command_line(&matches) {
         Ok(r) => r,
         Err(r) => {
             if let Error::CommandLine(r) = r {
@@ -155,20 +154,14 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
 }
 
 pub fn uu_app() -> Command {
-    Command::new(uucore::util_name())
+    let cmd = Command::new(uucore::util_name())
         .version(uucore::crate_version!())
-        .help_template(uucore::localized_help_template(uucore::util_name()))
         .about(translate!("chcon-about"))
         .override_usage(format_usage(&translate!("chcon-usage")))
-        .infer_long_args(true)
-        .disable_help_flag(true)
+        .infer_long_args(true);
+    uucore::clap_localization::configure_localized_command(cmd)
         .args_override_self(true)
-        .arg(
-            Arg::new(options::HELP)
-                .long(options::HELP)
-                .help(translate!("chcon-help-help"))
-                .action(ArgAction::Help),
-        )
+        .disable_help_flag(true)
         .arg(
             Arg::new(options::dereference::DEREFERENCE)
                 .long(options::dereference::DEREFERENCE)
@@ -183,6 +176,12 @@ pub fn uu_app() -> Command {
                 .help(translate!("chcon-help-no-dereference"))
                 .action(ArgAction::SetTrue),
         )
+        .arg(
+            Arg::new("help")
+                .long("help")
+                .help(translate!("help"))
+                .action(ArgAction::Help),
+        )
         .arg(
             Arg::new(options::preserve_root::PRESERVE_ROOT)
                 .long(options::preserve_root::PRESERVE_ROOT)
@@ -304,9 +303,7 @@ struct Options {
     files: Vec<PathBuf>,
 }
 
-fn parse_command_line(config: Command, args: impl uucore::Args) -> Result<Options> {
-    let matches = config.get_matches_from_localized(args);
-
+fn parse_command_line(matches: &ArgMatches) -> Result<Options> {
     let verbose = matches.get_flag(options::VERBOSE);
 
     let (recursive_mode, affect_symlink_referent) = if matches.get_flag(options::RECURSIVE) {

src/uu/chgrp/src/chgrp.rs

@@ -98,12 +98,12 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
 }
 
 pub fn uu_app() -> Command {
-    Command::new(uucore::util_name())
+    let cmd = Command::new(uucore::util_name())
         .version(uucore::crate_version!())
-        .help_template(uucore::localized_help_template(uucore::util_name()))
         .about(translate!("chgrp-about"))
         .override_usage(format_usage(&translate!("chgrp-usage")))
-        .infer_long_args(true)
+        .infer_long_args(true);
+    uucore::clap_localization::configure_localized_command(cmd)
         .disable_help_flag(true)
         .arg(
             Arg::new(options::HELP)