-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Gedding rid out a lot of 2>/dev/full abort #10646
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
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
I don't think so since this project does not recommend macros https://github.com/uutils/coreutils/blob/main/CONTRIBUTING.md#macros . |
|
A macro was used before, it's a utility macro used in many places and it unifies the code base with a single way of reporting errors. It also makes it easier to change the approach later and not have to go and fix every single use of |
|
yeah, it is probably easier to go this way |
|
@sylvestre I'm OK for both approach if we finally achieve compat binaries. |
|
But this macro PR does not cover everything. |
Most GNU does not catch I/O error of error messages.