Repository navigation
xargs: print the command line for -t as GNU does - #936
jayhemnani9910 wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #936 +/- ##
=======================================
Coverage 92.79% 92.80%
=======================================
Files 37 37
Lines 8038 8046 +8
Branches 403 403
=======================================
+ Hits 7459 7467 +8
Misses 437 437
Partials 142 142 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will not alter performance1 warning
Comparing |
xargs -tprints the Rust debug form of theCommand, which lists every environment variable:GNU prints the command and its arguments on one line, each quoted only where a shell would need it (
shell_escape_quoting_styleinprint_args):The line is now built from
get_program()andget_args()with uucore'sQuotingStyle::SHELL_ESCAPE. This enables uucore'squoting-stylefeature, which brings in the icu locale crates (16 new entries in Cargo.lock).New test
xargs_verbose_prints_the_command_linefails on main;cargo testpasses, clippy and fmt clean.