public inbox for linux-kbuild@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Michal Marek <mmarek@suse.cz>
Cc: Matthias Schniedermeyer <ms@citd.de>,
	Geert Uytterhoeven <geert@linux-m68k.org>,
	Sam Ravnborg <sam@ravnborg.org>,
	Sasha Levin <levinsasha928@gmail.com>,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Pekka Enberg <penberg@cs.helsinki.fi>,
	linux-kbuild@vger.kernel.org
Subject: Re: [PATCH] config: Add 'make kvmconfig'
Date: Wed, 2 Nov 2011 10:17:47 +0100	[thread overview]
Message-ID: <20111102091747.GH18903@elte.hu> (raw)
In-Reply-To: <4EB1084F.4020206@suse.cz>


* Michal Marek <mmarek@suse.cz> wrote:

> On 2.11.2011 09:41, Ingo Molnar wrote:
> > 
> > * Matthias Schniedermeyer <ms@citd.de> wrote:
> > 
> >> And AFAICS there currently is no "modconfig", only an 
> >> "allmodconfig" (and "localmodconfig")
> >>
> >> With no letters dropped it would be less ambiguous. make 
> >> modifyconfig=kvm
> > 
> > Yeah. Note that it's more like an 'add' operation, not a modify 
> > operation, thus:
> > 
> > 	make addconfig=kvm
> > 
> > or:
> > 
> > 	make addconfig kvm
> > 
> > would be intuitive and would work well too.
> > 
> > Sam, Michal, which one would you prefer? The latter might have the 
> > advantage that it's additive, enabling future usages like:
> > 
> > 	make addconfig kvm debug
> 
> The problem is that neither of these fit into the make command syntax:
> 
>     make addconfig kvm debug
> 
> tells make to build three different targets, and
> 
>     make addconfig=kvm
> 
> tells make to create a variable named addconfig with the value kvm 
> and build the default target. And BTW, you or someone else 
> reassured me earlier in the thread that this is going to be needed 
> for the kvm case solely, so why are you inventing something generic 
> now? :)

Because we are trying to find *something* that is both usable and 
which you guys are willing to ack.

So what's your suggestion? Is 'make kvmconfig' OK? It certainly is 
intuitive and useful to us and we could move on to more important 
things to discuss.

Thanks,

	Ingo

  reply	other threads:[~2011-11-02  9:22 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-27 12:07 [PATCH] config: Add 'make kvmconfig' Sasha Levin
2011-10-27 12:12 ` Pekka Enberg
2011-10-27 12:37   ` Ingo Molnar
2011-10-27 12:51     ` Michal Marek
2011-10-27 13:34       ` Ingo Molnar
2011-10-27 14:11       ` Pekka Enberg
2011-10-27 14:32         ` Michal Marek
2011-10-27 15:05           ` Arnaud Lacombe
2011-10-27 15:19         ` Arnaud Lacombe
2011-10-27 15:30           ` Pekka Enberg
2011-10-27 18:05 ` Rabin Vincent
2011-10-27 20:46   ` Sasha Levin
2011-10-27 19:34 ` Sam Ravnborg
2011-10-28  7:03   ` Ingo Molnar
2011-10-28  7:23     ` Sasha Levin
2011-10-28  7:39       ` Ingo Molnar
2011-10-28  7:47         ` Sasha Levin
2011-10-28  7:52           ` Ingo Molnar
2011-10-28  7:52     ` Geert Uytterhoeven
2011-10-28  7:54       ` Ingo Molnar
2011-10-28 12:08         ` Matthias Schniedermeyer
2011-10-28 12:29           ` Ingo Molnar
2011-10-28 13:05             ` Matthias Schniedermeyer
2011-11-02  8:41               ` Ingo Molnar
2011-11-02  9:07                 ` Michal Marek
2011-11-02  9:17                   ` Ingo Molnar [this message]
2011-11-02  9:24                     ` Michal Marek
2011-11-02  9:39                       ` Ingo Molnar
2011-11-02 10:06                         ` Pekka Enberg
2011-10-28  9:16       ` Sasha Levin

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=20111102091747.GH18903@elte.hu \
    --to=mingo@elte.hu \
    --cc=geert@linux-m68k.org \
    --cc=levinsasha928@gmail.com \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=ms@citd.de \
    --cc=penberg@cs.helsinki.fi \
    --cc=sam@ravnborg.org \
    --cc=torvalds@linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox