linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Walker <dwalker@codeaurora.org>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: linux-kbuild@vger.kernel.org, "Tony Lindgren" <tony@atomide.com>,
	"Nicolas Pitre" <nico@fluxnic.net>,
	linux-kernel@vger.kernel.org,
	"Linus Torvalds" <torvalds@linux-foundation.org>,
	"Russell King" <rmk@arm.linux.org.uk>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	linuxppc-dev@lists.ozlabs.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [RFC PATCH] Kconfig: Enable Kconfig fragments to be used for defconfig
Date: Tue, 13 Jul 2010 16:14:41 -0700	[thread overview]
Message-ID: <1279062881.4609.34.camel@c-dwalke-linux.qualcomm.com> (raw)
In-Reply-To: <20100713230352.6781.18644.stgit@angua>

On Tue, 2010-07-13 at 17:04 -0600, Grant Likely wrote:

> - I haven't figured out a way for the fragment to force an option to
>   be "n", or to set a value, for example "CONFIG_LOG_BUF_SHIFT=16".
>   This may require changing the syntax.
> - It still doesn't resolve dependencies.  A solver would help with this.
>   For the time being I work around the problem by running the generated
>   config through 'oldconfig' and looking for differences.  If the files
>   differ (ignoring comments and generateconfig_* options) after oldconfig,
>   then the <board>_defconfig target returns a failure.  (but leaves the
>   new .config intact so the user can resolve it with menuconfig).  This
>   way at least the user is told when a Kconfig fragment is invalid.

The solver would fix the whole issues with the defconfigs , we wouldn't
need this Kconfig change .. From my perspective we shouldn't be fooling
around with anything but the solver approach ..

It just doesn't feel like Kconfig was meant to do this, it feel like
somewhat of an abuse ..

Daniel

-- 
Sent by an consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

  parent reply	other threads:[~2010-07-13 23:27 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-13 23:04 [RFC PATCH] Kconfig: Enable Kconfig fragments to be used for defconfig Grant Likely
2010-07-13 23:11 ` Grant Likely
2010-07-13 23:14 ` Daniel Walker [this message]
2010-07-13 23:21   ` Grant Likely
2010-07-13 23:33     ` Daniel Walker
2010-07-14  0:07       ` Nicolas Pitre
2010-07-14 16:22         ` Daniel Walker
2010-07-16 23:49           ` Jamie Lokier
2010-07-19  5:20             ` Grant Likely
2010-07-16 16:03 ` Catalin Marinas
2010-07-16 17:57   ` Grant Likely
2010-07-16 18:07     ` Russell King - ARM Linux
2010-07-16 18:17       ` Linus Torvalds
2010-07-16 18:18       ` Grant Likely
2010-07-16 18:19     ` Nicolas Pitre
2010-07-16 18:21       ` Grant Likely
2010-07-16 18:31         ` Nicolas Pitre
2010-07-16 18:30       ` Russell King - ARM Linux
2010-07-16 18:40         ` Nicolas Pitre
2010-07-16 18:46           ` Linus Torvalds
2010-07-16 20:09             ` Catalin Marinas
2010-07-16 20:17               ` Grant Likely
2010-07-16 20:29                 ` Nicolas Pitre
2010-07-16 20:37                   ` Grant Likely
2010-07-16 20:44                 ` Catalin Marinas
2010-07-16 20:11             ` Arnd Bergmann
2010-07-16 18:52         ` Grant Likely
2010-07-16 20:01     ` Arnd Bergmann

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=1279062881.4609.34.camel@c-dwalke-linux.qualcomm.com \
    --to=dwalker@codeaurora.org \
    --cc=grant.likely@secretlab.ca \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=nico@fluxnic.net \
    --cc=rmk@arm.linux.org.uk \
    --cc=tony@atomide.com \
    --cc=torvalds@linux-foundation.org \
    --cc=u.kleine-koenig@pengutronix.de \
    /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;
as well as URLs for NNTP newsgroup(s).