Re: [kbuild-all] make[2]: *** No rule to make target 'tools/testing/nvdimm//config_check.o', needed by 'tools/testing/nvdimm//dax.o'.

From: Yilong Ren <yilongx.ren@intel.com>
To: Fengguang Wu <fengguang.wu@intel.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Dan Williams <dan.j.williams@intel.com>,
	kbuild-all@01.org, Johannes Thumshirn <jthumshirn@suse.de>
Subject: Re: [kbuild-all] make[2]: *** No rule to make target 'tools/testing/nvdimm//config_check.o', needed by 'tools/testing/nvdimm//dax.o'.
Date: Tue, 16 Aug 2016 10:33:27 +0800	[thread overview]
Message-ID: <20160816023327.GB13236@localhost.localdomain> (raw)
In-Reply-To: <20160816023014.GA17916@wfg-t540p.sh.intel.com>

On Tue, Aug 16, 2016 at 10:30:14AM +0800, Fengguang Wu wrote:
> >how about adding a function "exact_enable_module()" to assure use "m" ?
> >
> >diff --git a/lib/kconfig.sh b/lib/kconfig.sh
> >index 595dbfd..1502ce9 100644
> >--- a/lib/kconfig.sh
> >+++ b/lib/kconfig.sh
> >@@ -102,6 +102,7 @@ enable_testcase_config()
> >       do
> >               [[ $CONFIG =~ ^CONFIG_[A-Z0-9_]+=y$ ]] && enable_config ${CONFIG%=y}
> >+               [[ $CONFIG =~ ^CONFIG_[A-Z0-9_]+=m$ ]] && exact_enable_module ${CONFIG%=y}
> 
> s/y/m/
> 
> Otherwise looks good, thanks!

Got it,thanks, will do the patch.

-- 
Thanks
Ren Yilong

> 
> >               [[ $CONFIG =~ ^CONFIG_[A-Z0-9_]+[A-Z0-9]$ ]] && enable_module $CONFIG
> >               [[ $CONFIG =~ ^(CONFIG_[A-Z0-9_]+)=([0-9]+)$ ]] && set_config_to_value ${BASH_REMATCH[1]} ${BASH_REMATCH[2]}
> >       done
> >}