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&gt;; Qi, Fuli/斉 福利 <qi.fuli(a)jp.fujitsu.com&gt; 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&gt; > > Cc: linux-nvdimm <linux-nvdimm(a)lists.01.org&gt; > > 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&gt; > > 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&gt; > > > > Reviewed-by: Dan Williams <dan.j.williams(a)intel.com&gt; > > > > 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