From: Eric Sandeen <sandeen@xxxxxxxxxx> Documentation for these commands was a bit of a mess. 1) The help args were respecified in the _help() functions, overwriting the strings which had been set up in the _init functions as all other commands do. Worse, in the report case, they differed. 2) The -L/-U dump options were not present in either short help string. 3) The -L/-U dump options were not documented in the xfs_quota manpage. Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx> --- man/man8/xfs_quota.8 | 9 ++++++++- quota/report.c | 10 ++++------ 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/man/man8/xfs_quota.8 b/man/man8/xfs_quota.8 index 59e603f0..fd4562a1 100644 --- a/man/man8/xfs_quota.8 +++ b/man/man8/xfs_quota.8 @@ -393,7 +393,7 @@ option outputs the numeric ID instead of the name. The .B \-L and .B \-U -options specify lower and upper ID bounds to report on. If upper/lower +options specify lower and/or upper ID bounds to report on. If upper/lower bounds are specified, then by default only the IDs will be displayed in output; with the .B \-l @@ -558,6 +558,8 @@ report an error. [ .BR \-g " | " \-p " | " \-u ] [ +.BR \-L " | " \-U +] [ .B \-f .I file ] @@ -565,6 +567,11 @@ report an error. Dump out quota limit information for backup utilities, either to standard output (default) or to a .IR file . +The +.B \-L +and +.B \-U +options specify lower and/or upper ID bounds to dump. This is only the limits, not the usage information, of course. .HP .B restore diff --git a/quota/report.c b/quota/report.c index 6ac55490..2eb5b5a9 100644 --- a/quota/report.c +++ b/quota/report.c @@ -18,14 +18,14 @@ static cmdinfo_t report_cmd; static void dump_help(void) { - dump_cmd.args = _("[-g|-p|-u] [-f file]"); - dump_cmd.oneline = _("dump quota information for backup utilities"); printf(_( "\n" " create a backup file which contains quota limits information\n" " -g -- dump out group quota limits\n" " -p -- dump out project quota limits\n" " -u -- dump out user quota limits (default)\n" +" -L -- lower ID bound to dump\n" +" -U -- upper ID bound to dump\n" " -f -- write the dump out to the specified file\n" "\n")); } @@ -33,8 +33,6 @@ dump_help(void) static void report_help(void) { - report_cmd.args = _("[-bir] [-gpu] [-ahntlLNU] [-f file]"); - report_cmd.oneline = _("report filesystem quota information"); printf(_( "\n" " report used space and inodes, and quota limits, for a filesystem\n" @@ -757,7 +755,7 @@ report_init(void) dump_cmd.cfunc = dump_f; dump_cmd.argmin = 0; dump_cmd.argmax = -1; - dump_cmd.args = _("[-g|-p|-u] [-f file]"); + dump_cmd.args = _("[-g|-p|-u] [-LU] [-f file]"); dump_cmd.oneline = _("dump quota information for backup utilities"); dump_cmd.help = dump_help; dump_cmd.flags = CMD_FLAG_FOREIGN_OK; @@ -767,7 +765,7 @@ report_init(void) report_cmd.cfunc = report_f; report_cmd.argmin = 0; report_cmd.argmax = -1; - report_cmd.args = _("[-bir] [-gpu] [-ahnt] [-f file]"); + report_cmd.args = _("[-bir] [-gpu] [-ahntlLNU] [-f file]"); report_cmd.oneline = _("report filesystem quota information"); report_cmd.help = report_help; report_cmd.flags = CMD_FLAG_ONESHOT | CMD_FLAG_FOREIGN_OK; -- 2.33.1