Re: [PATCH] Fix fsck -C {fd} parsing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 24, 2015 at 09:01:10PM +0100, Stanislav Brabec wrote:
> Both fsck(8) and fsck --help documents progress bar redirection as
> -C {fd}
> 
> This did not work, and only -C{fd} and "-C {fd}" worked.

Perhaps rather than break the existing behavior it might be better to
make the documentation match reality.

Cheers,
dR

> ---
>  disk-utils/fsck.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/disk-utils/fsck.c b/disk-utils/fsck.c
> index 5842da4..817be97 100644
> --- a/disk-utils/fsck.c
> +++ b/disk-utils/fsck.c
> @@ -1471,7 +1471,7 @@ static void parse_argv(int argc, char *argv[])
>  					else
>  						goto next_arg;
>  				} else if (i+1 < argc && *argv[i+1] != '-') {	/* -C <fd> */
> -					progress_fd = string_to_int(argv[i]);
> +					progress_fd = string_to_int(argv[i+1]);
>  					if (progress_fd < 0)
>  						progress_fd = 0;
>  					else {
> -- 
> 2.3.3
> 
> -- 
> Best Regards / S pozdravem,
> 
> Stanislav Brabec
> software developer
> ---------------------------------------------------------------------
> SUSE LINUX, s. r. o.                          e-mail: sbrabec@xxxxxxx
> Lihovarská 1060/12                            tel: +49 911 7405384547
> 190 00 Praha 9                                 fax:  +420 284 084 001
> Czech Republic                                    http://www.suse.cz/
> PGP: 830B 40D5 9E05 35D8 5E27 6FA3 717C 209F A04F CD76
> --
> To unsubscribe from this list: send the line "unsubscribe util-linux" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux