diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-02-15 19:32:42 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-02-17 21:06:31 +0100 |
commit | 599c7c545f689f7d19a73238eecc69bf94fa6a74 (patch) | |
tree | 3d7cc8218ea1771ada8c36d094cda46a51c81291 /src/mount | |
parent | 79dbbb261d245b619a4d82c1cdf96f6a4310ee0d (diff) | |
download | systemd-599c7c545f689f7d19a73238eecc69bf94fa6a74.tar.gz |
tree-wide: add a helper to parse boolean optarg
This nicely covers the case when optarg is optional. The same parser can be
used when the option string passed to getopt_long() requires a parameter and
when it doesn't.
The error messages are made consistent.
Also fixes a log error c&p in --crash-reboot message.
Diffstat (limited to 'src/mount')
-rw-r--r-- | src/mount/mount-tool.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/mount/mount-tool.c b/src/mount/mount-tool.c index 8b6fc520c5..5a8e8b1143 100644 --- a/src/mount/mount-tool.c +++ b/src/mount/mount-tool.c @@ -23,6 +23,7 @@ #include "mount-util.h" #include "mountpoint-util.h" #include "pager.h" +#include "parse-argument.h" #include "parse-util.h" #include "path-util.h" #include "pretty-print.h" @@ -265,11 +266,9 @@ static int parse_argv(int argc, char *argv[]) { } case ARG_FSCK: - r = parse_boolean(optarg); + r = parse_boolean_argument("--fsck=", optarg, &arg_fsck); if (r < 0) - return log_error_errno(r, "Failed to parse --fsck= argument: %s", optarg); - - arg_fsck = r; + return r; break; case ARG_DESCRIPTION: |