-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Revert recent checksum CLI changes #10141
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
Conversation
|
some jobs are failing |
|
No need to revert everything. |
c9010ee to
b1edc49
Compare
I'm sorry it looks a bit severe, but I'm having a hard time making the difference between everything, partly because I wasn't following along in the last two weeks. I admit that this may not be the most optimal solution, but I really want to be done with getting rid of hashsum and code duplication already, so I'd rather revert too many things now and then reconsider all these changes once we have a common base to work on for all tools. |
|
GNU testsuite comparison: |
|
OK. I made every my *sum PR to draft. Let delete |
This PR reverts several changes that transitioned manual CLI argument parsing, at the expense of error messages explicitness and GNU behavior soundness.
This is preparatory work for cksum/hashsum merging, which will greatly simplify the maintenance of checksum-related utils