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

pr: add default values for -s and -S separator options (#10073)

Browse Source 
* pr: add default values for -s and -S separator options

* pr: fix -S default value to space, simplify test
This commit is contained in:
Chris Dryden
2026-01-07 16:06:52 +01:00
committed by GitHub
parent cc6414e5f5
commit ce8c8d57af
2 changed files with 20 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/uu/pr/src/pr.rs
+6 -2
View File
@@ -276,14 +276,18 @@ pub fn uu_app() -> Command {
.short('s')
.long(options::COLUMN_CHAR_SEPARATOR)
.help(translate!("pr-help-column-char-separator"))
.value_name("char"),
.value_name("char")
.num_args(0..=1)
.default_missing_value("\t"),
)
.arg(
Arg::new(options::COLUMN_STRING_SEPARATOR)
.short('S')
.long(options::COLUMN_STRING_SEPARATOR)
.help(translate!("pr-help-column-string-separator"))
.value_name("string"),
.value_name("string")
.num_args(0..=1)
.default_missing_value(" "),
)
.arg(
Arg::new(options::MERGE)
tests/by-util/test_pr.rs
+14
View File
@@ -627,3 +627,17 @@ fn test_page_header_width() {
let regex = Regex::new(&pattern).unwrap();
new_ucmd!().pipe_in("a").succeeds().stdout_matches(&regex);
}
#[test]
fn test_separator_options_default_values() {
// -s and -S without arguments should use default values (TAB and space)
// TODO: verify output matches GNU pr behavior
new_ucmd!()
.args(&["-t", "-2", "-s"])
.pipe_in("a\nb\n")
.succeeds();
new_ucmd!()
.args(&["-t", "-2", "-S"])
.pipe_in("a\nb\n")
.succeeds();
}