Skip to content

xargs: add --show-limits diagnostics#686

Open
kevinburke wants to merge 1 commit into
uutils:mainfrom
kevinburke:xargs-show-limits
Open

xargs: add --show-limits diagnostics#686
kevinburke wants to merge 1 commit into
uutils:mainfrom
kevinburke:xargs-show-limits

Conversation

@kevinburke
Copy link
Copy Markdown
Contributor

Among other things, Debian maintainers use the output of xargs --show-limits to set --max-chars appropriately for subsequent commands.

Accept GNU-compatible --show-limits and print command-line size diagnostics before reading input or building a command. The output includes the Maximum length line parsed by Debian maintainer scripts.

The regression test links to Debian python3.13's
libPVER-minimal.prerm.in, where 3.13.5-2+deb13u2 parses xargs --show-limits, and to GNU findutils 4.10.0's xargs/xargs.c, where the diagnostic text is emitted.

Reuse the existing system command-size calculation for both the default limiter and the diagnostic output, and add an integration test for the parsed limit.

Among other things, Debian maintainers use the output of xargs
--show-limits to set --max-chars appropriately for subsequent
commands.

Accept GNU-compatible --show-limits and print command-line size
diagnostics before reading input or building a command. The output includes
the Maximum length line parsed by Debian maintainer scripts.

The regression test links to Debian python3.13's
libPVER-minimal.prerm.in, where 3.13.5-2+deb13u2 parses xargs
--show-limits, and to GNU findutils 4.10.0's xargs/xargs.c, where the
diagnostic text is emitted.

Reuse the existing system command-size calculation for both the
default limiter and the diagnostic output, and add an integration test
for the parsed limit.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

❌ Patch coverage is 95.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.77%. Comparing base (c944cf7) to head (0ef2c9c).

Files with missing lines Patch % Lines
src/xargs/mod.rs 95.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #686      +/-   ##
==========================================
+ Coverage   91.76%   91.77%   +0.01%     
==========================================
  Files          31       31              
  Lines        6206     6240      +34     
  Branches      328      330       +2     
==========================================
+ Hits         5695     5727      +32     
- Misses        390      391       +1     
- Partials      121      122       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant