RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-30 04:27:45 +00:00
Code Issues Activity

sort: allow a sort mode to appear multiple times

Browse source 
A test case for this is `sort -k 2n,2n` which should be accepted, because `n` is
compatible with itself.

Resolves issue #4129.
This commit is contained in:
Terts Diepraam 2022-11-12 15:02:52 +01:00 • committed by Sylvestre Ledru
parent 8e1742f45a
commit 03c40b2db2
2 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/uu/sort/src/sort.rs
View file

@ -458,7 +458,7 @@ impl KeySettings {
}
fn set_sort_mode(&mut self, mode: SortMode) -> Result<(), String> {
if self.mode != SortMode::Default {
if self.mode != SortMode::Default && self.mode != mode {
return Err(format!(
"options '-{}{}' are incompatible",
self.mode.get_short_name().unwrap(),

5
tests/by-util/test_sort.rs
View file

@ -1156,3 +1156,8 @@ fn test_tmp_files_deleted_on_sigint() {
// `sort` should have deleted the temporary directory again.
assert!(read_dir(at.plus("tmp_dir")).unwrap().next().is_none());
}
#[test]
fn test_same_sort_mode_twice() {
new_ucmd!().args(&["-k", "2n,2n", "empty.txt"]).succeeds();
}