RE: [PATCH v6 1/4] ndctl, util: add OPTION_FILENAME to parse_opt_type
-----Original Message-----
From: Verma, Vishal L [mailto:vishal.l.verma@intel.com]
Sent: Wednesday, June 13, 2018 7:45 AM
To: Williams, Dan J <dan.j.williams(a)intel.com>; Qi, Fuli/斉 福利
<qi.fuli(a)jp.fujitsu.com>
Cc: linux-nvdimm(a)lists.01.org
Subject: Re: [PATCH v6 1/4] ndctl, util: add OPTION_FILENAME to
parse_opt_type
On Wed, 2018-05-16 at 02:42 +0000, Qi, Fuli wrote:
> > -----Original Message-----
> > From: Dan Williams [mailto:dan.j.williams@intel.com]
> > Sent: Saturday, May 12, 2018 1:42 AM
> > To: Qi, Fuli/斉 福利 <qi.fuli(a)jp.fujitsu.com>
> > Cc: linux-nvdimm <linux-nvdimm(a)lists.01.org>
> > Subject: Re: [PATCH v6 1/4] ndctl, util: add OPTION_FILENAME to
> > parse_opt_type
> >
> > On Sun, May 6, 2018 at 10:09 PM, QI Fuli <qi.fuli(a)jp.fujitsu.com>
> > wrote:
> > > This patch borrows the OPTION_FILENAME from git to ndctl to make
> > > sure filename is correct. Some related refactoring is also included:
> > > - adds parse_options_prefix() interface
> > > - moves is_absolute from util/help.c to util/util.c
> > > - adds a new file util/abspath.c
> > >
> > > Signed-off-by: QI Fuli <qi.fuli(a)jp.fujitsu.com>
> >
> > Reviewed-by: Dan Williams <dan.j.williams(a)intel.com>
> >
>
> Since this patch is independent from the others, could you please
> merge this one first?
Hi Qi,
Yes I have applied the first patch, and it is now in the pending branch.
Thanks,
-Vishal
Thank you very much.
Qi