uutils/coreutils
0
0
Fork 0
mirror of https://github.com/uutils/coreutils.git synced 2026-06-10 15:48:22 -07:00
Code Issues Packages Projects Releases Wiki Activity

Port factor for translation

Browse Source
This commit is contained in:
Sylvestre Ledru
2025-08-13 14:15:56 +02:00
parent 4b87d5abca
commit 37c633ecc6
2 changed files with 23 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/uucore/src/lib/mods/clap_localization.rs
+10 -15
View File
@@ -125,11 +125,15 @@ pub fn handle_clap_error_with_exit_code(err: Error, util_name: &str, exit_code:
_ => 1,
};
// UnknownArgument gets special handling for suggestions, but should still show simple help
// For UnknownArgument, we need to preserve clap's built-in tips (like using -- for values)
// while still allowing localization of the main error message
let rendered_str = err.render().to_string();
let _lines: Vec<&str> = rendered_str.lines().collect();
if let Some(invalid_arg) = err.get(ContextKind::InvalidArg) {
let arg_str = invalid_arg.to_string();
// Get the uncolored words from common strings with fallbacks
// Get localized error word with fallback
let error_word = {
let translated = translate!("common-error");
if translated == "common-error" {
@@ -138,18 +142,9 @@ pub fn handle_clap_error_with_exit_code(err: Error, util_name: &str, exit_code:
translated
}
};
let tip_word = {
let translated = translate!("common-tip");
if translated == "common-tip" {
"tip".to_string()
} else {
translated
}
};
let colored_arg = maybe_colorize(&arg_str, Color::Yellow);
let colored_error_word = maybe_colorize(&error_word, Color::Red);
let colored_tip_word = maybe_colorize(&tip_word, Color::Green);
// Print main error message with fallback
let error_msg = {
@@ -173,6 +168,8 @@ pub fn handle_clap_error_with_exit_code(err: Error, util_name: &str, exit_code:
// Show suggestion if available
let suggestion = err.get(ContextKind::SuggestedArg);
if let Some(suggested_arg) = suggestion {
let tip_word = translate!("common-tip");
let colored_tip_word = maybe_colorize(&tip_word, Color::Green);
let colored_suggestion =
maybe_colorize(&suggested_arg.to_string(), Color::Green);
let suggestion_msg = {
@@ -203,9 +200,8 @@ pub fn handle_clap_error_with_exit_code(err: Error, util_name: &str, exit_code:
}
}
// Show usage information for unknown arguments but use simple --help format
// Try to get usage information from localization with fallback
let usage_key = format!("{}-usage", util_name);
// Show usage information for unknown arguments
let usage_key = format!("{util_name}-usage");
let usage_text = translate!(&usage_key);
let formatted_usage = crate::format_usage(&usage_text);
let usage_label = {
@@ -218,7 +214,6 @@ pub fn handle_clap_error_with_exit_code(err: Error, util_name: &str, exit_code:
};
eprintln!("{}: {}", usage_label, formatted_usage);
eprintln!();
// Use simple --help format for GNU test compatibility
eprintln!("For more information, try '--help'.");
std::process::exit(exit_code);
tests/by-util/test_factor.rs
+13
View File
@@ -28,6 +28,19 @@ fn test_invalid_arg() {
new_ucmd!().arg("--definitely-invalid").fails_with_code(1);
}
#[test]
fn test_invalid_negative_arg_shows_tip() {
// Test that factor shows a tip when given an invalid negative argument
// This replicates the GNU test issue where "-1" was interpreted as an invalid option
new_ucmd!()
.arg("-1")
.fails()
.code_is(1)
.stderr_contains("unexpected argument '-1' found")
.stderr_contains("tip: to pass '-1' as a value, use '-- -1'")
.stderr_contains("Usage: factor");
}
#[test]
fn test_valid_arg_exponents() {
new_ucmd!().arg("-h").succeeds();