-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
mdconfig: Print options when listing devices #1210
Conversation
31b8671
to
c55d997
Compare
sbin/mdconfig/mdconfig.c
Outdated
fprintf(stderr, "\t\t verify}\n"); | ||
fprintf(stderr, "\t\toption = {async, cache, cluster, compress,\n"); | ||
fprintf(stderr, "\t\t force, mustdealloc, readonly, ro,\n"); | ||
fprintf(stderr, "\t\t reserve, verify}\n"); | ||
fprintf(stderr, "\t\tsize = %%d (512 byte blocks), %%db (B),\n"); | ||
fprintf(stderr, "\t\t %%dk (kB), %%dm (MB), %%dg (GB), \n"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should delete this space after the , and before the \n
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should delete this space after the , and before the \n
Fixed, thanks!
Otherwise, I like this. |
c55d997
to
4c847fa
Compare
PR: 278624 Reviewed by: imp Pull Request: freebsd#1210
Thanks for this! Please close the Bugzilla PR now that this is in. |
@ricardobranco777 what is the purpose of the |
You're right. It must be |
Doh! Looks like I missed seeing that. Thanks for fixing. |
Currently, we have no way of seeing options set to md(4) devices.
Also fix usage string (missing
async
option).Example:
# mdconfig -a -f /root/backup.iso -o readonly -o verify
Before:
After:
Fixes https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=278624