Re: [f2fs-dev] [PATCH 2/2] fsck.f2fs: support -p without argument

2017-01-20 Thread Jaegeuk Kim
On 01/20, Sheng Yong wrote: > Hi Jaegeuk, > > On 1/20/2017 5:47 PM, Jaegeuk Kim wrote: > > On 01/20, Sheng Yong wrote: > [..] > >>> > >>> if (!strcmp("fsck.f2fs", prog)) { > >>> - const char *option_string = "ad:fp:t"; > >>> + const char *option_string = ":a:d:f:p:t:"; > >> I

Re: [f2fs-dev] [PATCH 2/2] fsck.f2fs: support -p without argument

2017-01-20 Thread Sheng Yong
Hi Jaegeuk, On 1/20/2017 5:47 PM, Jaegeuk Kim wrote: > On 01/20, Sheng Yong wrote: [..] >>> >>> if (!strcmp("fsck.f2fs", prog)) { >>> - const char *option_string = "ad:fp:t"; >>> + const char *option_string = ":a:d:f:p:t:"; >> I think there is no need to modify options

Re: [f2fs-dev] [PATCH 2/2] fsck.f2fs: support -p without argument

2017-01-20 Thread Sheng Yong
Hi Jaegeuk, On 1/20/2017 7:19 AM, Jaegeuk Kim wrote: > Hi Sheng Yong, > > I tested this, but failed on -p with arguments. Sorry, this may because double colon in optsting can only be used at some Unix-like distributions :( > > Could you take a look at this change? I tested this patch, it

Re: [f2fs-dev] [PATCH 2/2] fsck.f2fs: support -p without argument

2017-01-19 Thread Jaegeuk Kim
Hi Sheng Yong, I tested this, but failed on -p with arguments. Could you take a look at this change? >From e558ebf31a35619d5625927b50be0c9feef1feac Mon Sep 17 00:00:00 2001 From: Jaegeuk Kim Date: Thu, 19 Jan 2017 11:03:41 +0800 Subject: [PATCH] fsck.f2fs: support -p

[f2fs-dev] [PATCH 2/2] fsck.f2fs: support -p without argument

2017-01-18 Thread Sheng Yong
This patch allows fsck run -p without argument. So we could use -p as -p, -p0, and -p1. '-p' and '-p0' have the same meaning as '-a'. '-p1' checks more meta data than '-a'. Reported-by: KARBOWSKI Piotr Signed-off-by: Sheng Yong --- fsck/main.c