Skip to content

hashsum: fix help text to show actual utility name#8650

Merged
sylvestre merged 2 commits into
uutils:mainfrom
AnarchistHoneybun:bug/hashsum-help-name
Sep 21, 2025
Merged

hashsum: fix help text to show actual utility name#8650
sylvestre merged 2 commits into
uutils:mainfrom
AnarchistHoneybun:bug/hashsum-help-name

Conversation

@AnarchistHoneybun

Copy link
Copy Markdown
Contributor
  • Fix all hashsum utilities (md5sum, b2sum, sha256sum, etc.) to show correct utility name in help
  • Previously showed 'Usage: hashsum --' instead of 'Usage: md5sum [OPTION]...'

Fixes #8614

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

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

Comment thread src/uu/hashsum/src/hashsum.rs Outdated
@sylvestre

Copy link
Copy Markdown
Contributor

Could you please add a test to make sure we don't regress in the future? Thanks

@AnarchistHoneybun AnarchistHoneybun force-pushed the bug/hashsum-help-name branch 3 times, most recently from a10c14e to 7ed73f7 Compare September 17, 2025 07:42
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

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

@AnarchistHoneybun

Copy link
Copy Markdown
Contributor Author

@sylvestre is there a plan to merge this? Just worried about having to add any changes from main to this branch as well if they're kept apart long

Comment thread src/uu/hashsum/locales/en-US.ftl Outdated
Fix all hashsum utilities (md5sum, b2sum, sha256sum, etc.) to display
their actual name in help output instead of generic 'hashsum --<digest>'.

Override help_template and usage for specific utilities while preserving
the original hashsum behavior for the generic multicall case.

Add test to verify help output shows correct utility names and prevent
future regression.

Fixes uutils#8614
@codspeed-hq

codspeed-hq Bot commented Sep 21, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #8650 will create unknown performance changes

Comparing AnarchistHoneybun:bug/hashsum-help-name (58a04bf) with main (0ccc67f)

Summary

⚠️ No benchmarks were detected in both the base of the PR and the PR.\

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

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

@sylvestre sylvestre merged commit 7bca02c into uutils:main Sep 21, 2025
98 checks passed
@sylvestre

Copy link
Copy Markdown
Contributor

thanks! :)

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.

0.2.2: All hashsum (b2sum,md5sum etc.) programs report hashsum in --help switch on multicall binary

2 participants