-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix docs hashsumS and test #7936
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Hi With #7936 Usage: target/debug/coreutils md5sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha1sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha224sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha256sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha384sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha512sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha3sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha3-224sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha3-256sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha3-384sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils sha3-512sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils shake128sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils shake256sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils b2sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils b3sum [OPTIONS]... [FILE]...
Usage: target/debug/coreutils hashsum --<digest> [OPTIONS]... [FILE]... |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
Can the testsuite currently test these behaviors @sylvestre ? |
|
@Its-Just-Nans yeah, maybe add tests for this to make sure we don't regress? thanks @RenjiSann dunno, sorry :) |
done |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
@sylvestre I think now it's ready Note that it seems that some binaries where missing in |
|
#7936 (comment) Non GNU binary names are dropped. |
Thanks you, make sense. Fixed the tests |
|
GNU testsuite comparison: |
|
@RenjiSann @oech3 ok to merge this? thanks |
|
Non GNU binaries are not dropped yet |
yep, and |
|
@Its-Just-Nans Would you split PR as |
I'm not sure to understand what that "split PR" means. Feel free to edit it! |
|
Should we close this as |
This still contains a fix for the man |
|
@oech3 I've updated my MR |
|
GNU testsuite comparison: |
Fixing the mdbook docs for