Skip to content

Commit 49c7d72

Browse files
committed
ps: Sort before dedup
dedup_by() only deduplicates consecutively identical elements, so `ps -A` is returning some processes twice.
1 parent 9093e4e commit 49c7d72

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/uu/ps/src/ps.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
4242
proc_infos.extend(collector::process_collector(&matches, &snapshot));
4343
proc_infos.extend(collector::session_collector(&matches, &snapshot));
4444

45+
proc_infos.sort_by(|a, b| a.borrow().pid.cmp(&b.borrow().pid));
4546
proc_infos.dedup_by(|a, b| a.borrow().pid == b.borrow().pid);
4647

4748
sorting::sort(&mut proc_infos, &matches);

0 commit comments

Comments
 (0)