[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 05/23] qemu-img: simplify --repair error message
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 05/23] qemu-img: simplify --repair error message |
Date: |
Tue, 20 Feb 2024 17:40:49 +0000 |
User-agent: |
Mutt/2.2.12 (2023-09-09) |
On Sat, Feb 10, 2024 at 12:22:26AM +0300, Michael Tokarev wrote:
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
> qemu-img.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-img.c b/qemu-img.c
> index 3ae07bfae0..ad7fa033b1 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -843,8 +843,8 @@ static int img_check(const img_cmd_t *ccmd, int argc,
> char **argv)
> } else if (!strcmp(optarg, "all")) {
> fix = BDRV_FIX_LEAKS | BDRV_FIX_ERRORS;
> } else {
> - error_exit(ccmd, "Unknown option value for -r "
> - "(expecting 'leaks' or 'all'): %s", optarg);
> + error_exit(ccmd,
> + "--repair expects 'leaks' or 'all' not '%s'",
> optarg);
> }
Should we say '--repair/-r expects...' since we don't know which the
user passed
Either way
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [PATCH 00/23] qemu-img: refersh options and --help handling, Michael Tokarev, 2024/02/09
- [PATCH 01/23] qemu-img: pass current cmd info into command handlers, Michael Tokarev, 2024/02/09
- [PATCH 02/23] qemu-img: refresh options/--help for "create" subcommand, Michael Tokarev, 2024/02/09
- [PATCH 03/23] qemu-img: factor out parse_output_format() and use it in the code, Michael Tokarev, 2024/02/09
- [PATCH 04/23] qemu-img: refresh options/--help for "check" command, Michael Tokarev, 2024/02/09
- [PATCH 05/23] qemu-img: simplify --repair error message, Michael Tokarev, 2024/02/09
- Re: [PATCH 05/23] qemu-img: simplify --repair error message,
Daniel P . Berrangé <=
- [PATCH 06/23] qemu-img: refresh options/--help for "commit" command, Michael Tokarev, 2024/02/09
- [PATCH 07/23] qemu-img: refresh options/--help for "compare" command, Michael Tokarev, 2024/02/09
- [PATCH 08/23] qemu-img: refresh options/--help for "convert" command, Michael Tokarev, 2024/02/09
- [PATCH 09/23] qemu-img: refresh options/--help for "info" command, Michael Tokarev, 2024/02/09
- [PATCH 10/23] qemu-img: refresh options/--help for "map" command, Michael Tokarev, 2024/02/09
- [PATCH 11/23] qemu-img: allow specifying -f fmt for snapshot subcommand, Michael Tokarev, 2024/02/09
- [PATCH 13/23] qemu-img: refresh options/--help for "snapshot" command, Michael Tokarev, 2024/02/09
- [PATCH 14/23] qemu-img: refresh options/--help for "rebase" command, Michael Tokarev, 2024/02/09