Skip to content

Conversation

@mattsu2020
Copy link
Contributor

@mattsu2020 mattsu2020 commented Jan 5, 2026

Summary

  • Extend CmdResult to carry execution metadata (args/env/cwd/stdin).
  • Add GNU comparison helper (matches_gnu) on Unix and propagate metadata through UChild/gnu_cmd_result.

Notes

Added #[allow(clippy::too_many_arguments)] to the CmdResult::new function in tests/uutests/src/lib/util.rs to suppress the Clippy warning, as the function legitimately requires multiple parameters for its intended functionality.
…dability

Refactor the chained method calls to break across multiple lines, improving code formatting and adherence to Rust style guidelines without altering functionality.
@sylvestre
Copy link
Contributor

sorry, i don't want to have a way in uutests to compare with gnu output
just run regular test with the expected output

to compare with gnu, extending the differential fuzzer will be better
see
https://github.com/uutils/coreutils/blob/main/fuzz/fuzz_targets/fuzz_sort.rs#L77

@mattsu2020
Copy link
Contributor Author

sorry, i don't want to have a way in uutests to compare with gnu output just run regular test with the expected output

to compare with gnu, extending the differential fuzzer will be better see https://github.com/uutils/coreutils/blob/main/fuzz/fuzz_targets/fuzz_sort.rs#L77

OK close this

@mattsu2020 mattsu2020 closed this Jan 5, 2026
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants