From: Michal Marek <mmarek@suse.cz>
To: Ben Gardiner <bengardiner@nanometrics.ca>
Cc: Roman Zippel <zippel@linux-m68k.org>,
linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] kbuild: fix interaction of CONFIG_IKCONFIG and KCONFIG_CONFIG
Date: Tue, 14 Dec 2010 16:29:23 +0100 [thread overview]
Message-ID: <4D078D53.4040700@suse.cz> (raw)
In-Reply-To: <1290797770-22877-1-git-send-email-bengardiner@nanometrics.ca>
On 26.11.2010 19:56, Ben Gardiner wrote:
> Change the use of .config in kernel/Makefile to $(KCONFIG_CONFIG).
>
> Currently, if you try to build a kernel with KCONFIG_CONFIG set (to a value
> not equal to .config) and the config file pointed to by KCONFIG_CONFIG sets
> CONFIG_IKCONFIG then the build will fail with:
>
> make[1]: *** No rule to make target `.config', needed by `kernel/config_data.gz'. Stop.
>
> I think this has been present since the introduction of KCONFIG_CONFIG
> in 14cdd3c402bf7c66f0bcd76e290f0770a54a4b21.
>
> Signed-off-by: Ben Gardiner<bengardiner@nanometrics.ca>
> CC: Roman Zippel<zippel@linux-m68k.org>
> CC: Michal Marek<mmarek@suse.cz>
>
> ---
> based on 698fd6a2c3ca05ec796072defb5c415289a86cdc of
> git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
>
> kernel/Makefile | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/Makefile b/kernel/Makefile
> index 0b5ff08..33e0a39 100644
> --- a/kernel/Makefile
> +++ b/kernel/Makefile
> @@ -121,7 +121,7 @@ $(obj)/configs.o: $(obj)/config_data.h
> # config_data.h contains the same information as ikconfig.h but gzipped.
> # Info from config_data can be extracted from /proc/config*
> targets += config_data.gz
> -$(obj)/config_data.gz: .config FORCE
> +$(obj)/config_data.gz: $(KCONFIG_CONFIG) FORCE
> $(call if_changed,gzip)
>
> quiet_cmd_ikconfiggz = IKCFG $@
The problem is that KCONFIG_CONFIG is not exported in the top-level
Makefile. If you want to use it in other Makefiles, you need to add the
export.
Michal
next prev parent reply other threads:[~2010-12-14 15:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-26 18:56 [PATCH] kbuild: fix interaction of CONFIG_IKCONFIG and KCONFIG_CONFIG Ben Gardiner
2010-11-26 18:56 ` Ben Gardiner
2010-12-14 15:29 ` Michal Marek [this message]
2010-12-14 16:39 ` [PATCH v2] " Ben Gardiner
2010-12-14 16:39 ` Ben Gardiner
2010-12-14 22:18 ` Michal Marek
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=4D078D53.4040700@suse.cz \
--to=mmarek@suse.cz \
--cc=bengardiner@nanometrics.ca \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=zippel@linux-m68k.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.