Skip to content

shred: ensure deterministic pass sequence compatibility with reference implementation#9317

Merged
cakebaker merged 3 commits into
uutils:mainfrom
sylvestre:shred-passes
Dec 22, 2025
Merged

shred: ensure deterministic pass sequence compatibility with reference implementation#9317
cakebaker merged 3 commits into
uutils:mainfrom
sylvestre:shred-passes

Conversation

@sylvestre

Copy link
Copy Markdown
Contributor

should fix tests/shred/shred-passes.sh

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/shred/shred-passes is no longer failing!

@oech3

This comment was marked as resolved.

@sylvestre sylvestre force-pushed the shred-passes branch 2 times, most recently from 7d9a9ce to 4d5eed9 Compare November 18, 2025 06:41
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/shred/shred-passes is no longer failing!

Comment thread src/uu/shred/src/shred.rs Outdated
@oech3

oech3 commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/runcon/runcon-compute. tests/runcon/runcon-compute is passing on 'main'. Maybe you have to rebase?
Congrats! The gnu test tests/runcon/runcon-no-reorder is no longer failing!
Congrats! The gnu test tests/shred/shred-passes is no longer failing!

@sylvestre sylvestre requested a review from cakebaker December 20, 2025 10:42
@cakebaker cakebaker merged commit 87ba3af into uutils:main Dec 22, 2025
127 of 128 checks passed
@cakebaker

Copy link
Copy Markdown
Contributor

Congrats! The gnu test tests/shred/shred-passes is no longer failing!

Kudos!

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.

3 participants