Skip to content

Add --debug flag to cksum#9088

Merged
RenjiSann merged 2 commits into
uutils:mainfrom
naoNao89:fix-cksum-debug-flag
Nov 26, 2025
Merged

Add --debug flag to cksum#9088
RenjiSann merged 2 commits into
uutils:mainfrom
naoNao89:fix-cksum-debug-flag

Conversation

@naoNao89

Copy link
Copy Markdown
Contributor

This PR implements the --debug flag for cksum, split from the original PR #8948.

What's included:

  • --debug CLI argument with help text
  • Localization for English and French
  • Tests covering single file, multiple files, and algorithm combinations

@codspeed-hq

codspeed-hq Bot commented Oct 30, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #9088 will not alter performance

Comparing naoNao89:fix-cksum-debug-flag (43f3eb3) with main (b2d1117)

Summary

✅ 126 untouched
⏩ 6 skipped1

Footnotes

  1. 6 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@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)
Congrats! The gnu test tests/cksum/cksum is no longer failing!

@cakebaker cakebaker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add a hint to the help text that the debug functionality is not yet implemented? To make it visible to the user that it is a no-op. Otherwise the PR looks good.

@naoNao89

Copy link
Copy Markdown
Contributor Author

@cakebaker Updated! ✅

Added the clarification to the help text as requested:

English:

--debug    indicate which implementation used (not yet implemented)

French:

--debug    indiquer quelle implémentation est utilisée (pas encore implémenté)

This follows the same pattern used by other utilities in the codebase (cp, mkdir, split) for indicating unimplemented features.

The test still passes:

test test_cksum::test_debug_flag ... ok

@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)
Congrats! The gnu test tests/cksum/cksum is no longer failing!

@sylvestre

Copy link
Copy Markdown
Contributor

Why not implementing it?

@naoNao89 naoNao89 marked this pull request as draft October 30, 2025 11:25
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/cksum/cksum is no longer failing!

@naoNao89 naoNao89 marked this pull request as ready for review October 30, 2025 18:09
Comment thread tests/by-util/test_cksum.rs
@github-actions

github-actions Bot commented Nov 1, 2025

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/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

@naoNao89 naoNao89 force-pushed the fix-cksum-debug-flag branch 3 times, most recently from e99e61e to c4276f6 Compare November 1, 2025 13:46
@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

1 similar comment
@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions

github-actions Bot commented Nov 2, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@github-actions

github-actions Bot commented Nov 2, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

1 similar comment
@github-actions

github-actions Bot commented Nov 2, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions

github-actions Bot commented Nov 2, 2025

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

Comment thread Cargo.toml Outdated
@naoNao89 naoNao89 force-pushed the fix-cksum-debug-flag branch 3 times, most recently from c3ba054 to 82cb1e1 Compare November 24, 2025 22:46
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@naoNao89 naoNao89 requested a review from RenjiSann November 24, 2025 23:36
Comment thread tests/by-util/test_cksum.rs Outdated
Comment thread tests/fixtures/cksum/base64/crc32b_single_file.expected
Comment thread src/uu/cksum/src/cksum.rs
@naoNao89 naoNao89 force-pushed the fix-cksum-debug-flag branch 2 times, most recently from f94e999 to 5ba81e1 Compare November 25, 2025 02:37
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@naoNao89 naoNao89 requested a review from RenjiSann November 25, 2025 04:48
Comment thread tests/by-util/test_cksum.rs Outdated
@naoNao89 naoNao89 force-pushed the fix-cksum-debug-flag branch from 5ba81e1 to 19ea339 Compare November 25, 2025 17:57
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@naoNao89 naoNao89 force-pushed the fix-cksum-debug-flag branch from 19ea339 to 77d2dd3 Compare November 25, 2025 22:56
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@RenjiSann RenjiSann merged commit d655eed into uutils:main Nov 26, 2025
128 checks passed
@RenjiSann

Copy link
Copy Markdown
Collaborator

Thanks !

@naoNao89 naoNao89 deleted the fix-cksum-debug-flag branch November 26, 2025 11:58
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.

4 participants