On Wed, 15 Sep 2010 16:01:32 +0900, Jiro SEKIBA wrote:
> Hi,
>
> Change the error message of nilfs-tune when user doesn't have
> a permission to access the device to show the cause of error clearly.
>
> differences from v1
> - moved the error string behind the primary message.
>
> Signed-off-by: Jiro SEKIBA <[email protected]>
Applied.
Thanks,
Ryusuke Konishi
> ---
> sbin/nilfs-tune/nilfs-tune.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/sbin/nilfs-tune/nilfs-tune.c b/sbin/nilfs-tune/nilfs-tune.c
> index d99ce46..e80caa4 100644
> --- a/sbin/nilfs-tune/nilfs-tune.c
> +++ b/sbin/nilfs-tune/nilfs-tune.c
> @@ -399,10 +399,12 @@ int modify_nilfs(char *device, struct
> nilfs_tune_options *opts)
> struct nilfs_super_block *sbp;
> __u64 features;
>
> + errno = 0;
> devfd = open(device, opts->flags);
>
> if (devfd == -1) {
> - fprintf(stderr, "%s: cannot open NILFS\n", device);
> + fprintf(stderr, "cannot open device %s: %s\n",
> + device, strerror(errno));
> ret = EXIT_FAILURE;
> goto out;
> }
> --
> 1.5.6.5
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
