From 56f4e809fd7043b1451defee203cbee776c085d5 Mon Sep 17 00:00:00 2001 From: Hanif Bin Ariffin Date: Thu, 29 Jul 2021 21:36:13 +0800 Subject: [PATCH] Silently accepts ---presume-input-tty For whatever reason, the following is equivalent, cargo run -- rm --presume-input-tty cargo run -- rm ---presume-input-tty cargo run -- rm -----presume-input-tty cargo run -- rm ---------presume-input-tty Signed-off-by: Hanif Bin Ariffin --- src/uu/rm/src/rm.rs | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/src/uu/rm/src/rm.rs b/src/uu/rm/src/rm.rs index 259d1ab39..1fdc59371 100644 --- a/src/uu/rm/src/rm.rs +++ b/src/uu/rm/src/rm.rs @@ -49,6 +49,7 @@ static OPT_PROMPT_MORE: &str = "prompt-more"; static OPT_RECURSIVE: &str = "recursive"; static OPT_RECURSIVE_R: &str = "recursive_R"; static OPT_VERBOSE: &str = "verbose"; +static PRESUME_INPUT_TTY: &str = "presume-input-tty"; static ARG_FILES: &str = "files"; @@ -206,6 +207,17 @@ pub fn uu_app() -> App<'static, 'static> { .long(OPT_VERBOSE) .help("explain what is being done") ) + // From the GNU source code: + // This is solely for testing. + // Do not document. + // It is relatively difficult to ensure that there is a tty on stdin. + // Since rm acts differently depending on that, without this option, + // it'd be harder to test the parts of rm that depend on that setting. + .arg( + Arg::with_name(PRESUME_INPUT_TTY) + .long(PRESUME_INPUT_TTY) + .hidden(true) + ) .arg( Arg::with_name(ARG_FILES) .multiple(true)