The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Fengguang Wu <fengguang.wu@intel.com>
To: Yilong Ren <yilongx.ren@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:30:14 +0800	[thread overview]
Message-ID: <20160816023014.GA17916@wfg-t540p.sh.intel.com> (raw)
In-Reply-To: <20160816021929.GE6835@localhost.localdomain>

>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!

>                [[ $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
> }

  reply	other threads:[~2016-08-16  2:30 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-15 10:03 make[2]: *** No rule to make target 'tools/testing/nvdimm//config_check.o', needed by 'tools/testing/nvdimm//dax.o' kbuild test robot
2016-08-16  0:58 ` Dan Williams
2016-08-16  1:26   ` [kbuild-all] " Fengguang Wu
2016-08-16  1:30     ` Dan Williams
2016-08-16  1:36       ` Fengguang Wu
2016-08-16  1:41       ` Fengguang Wu
2016-08-16  1:47         ` Yilong Ren
2016-08-16  1:58           ` Fengguang Wu
2016-08-16  2:19             ` Yilong Ren
2016-08-16  2:30               ` Fengguang Wu [this message]
2016-08-16  2:33                 ` Yilong Ren
2016-08-16  1:48         ` Dan Williams
2016-08-16  1:41       ` Yilong Ren
2016-08-22 12:26       ` Yilong Ren
2016-08-23 23:42         ` Dan Williams
2016-08-24  2:47           ` Fengguang Wu
2016-08-24 14:59             ` Dan Williams
2016-08-25  0:09               ` Fengguang Wu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160816023014.GA17916@wfg-t540p.sh.intel.com \
    --to=fengguang.wu@intel.com \
    --cc=dan.j.williams@intel.com \
    --cc=jthumshirn@suse.de \
    --cc=kbuild-all@01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=yilongx.ren@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox