All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Landley <rob@landley.net>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: linux-kernel@vger.kernel.org, Sam Ravnborg <sam@ravnborg.org>
Subject: Re: Quick and dirty miniconfig howto, with feature suggestions.
Date: Sat, 19 Nov 2005 23:54:18 -0600	[thread overview]
Message-ID: <200511192354.19108.rob@landley.net> (raw)
In-Reply-To: <Pine.LNX.4.61.0511192338300.1609@scrub.home>

On Saturday 19 November 2005 21:08, Roman Zippel wrote:
> Hi,
>
> On Thu, 17 Nov 2005, Rob Landley wrote:
> > 1) Add a "make miniconfig" which works like allnoconfig but A) takes
> > mini.config as its' default name, B) redirects stdout to /dev/null to
> > make it easier to spot typoed symbols, C) aborts (exits with an error,
> > does not write new .config) if mini.config isn't found or if it contains
> > an unrecognized symbol.
>
> I think I better make allnoconfig silent (unless with V=1 or something),
> which makes it your miniconfig already almost like allnoconfig.

That would be an improvement, but I think from a user interface perspective 
it's slightly confusing to overload allnoconfig.  (It's not a big issue 
either way, though.)

The output redirecton could actually be done trivially in the makefile without 
touching the config code, and the other changes I proposed are fairly small.    
I have a todo item to whip up a "make miniconfig" patch that adds a new 
target with all three of the small behavior changes.  I can submit that for 
consideration later this evening...

> I'm not quite sure about aborting there are other error possibilities
> (e.g. new dependencies), so you never quite can trust the error value
> anyway.

In theory, something based on allnoconfig shouldn't care about the previous 
state of the .config file, so how is a dependency "new"?  (Could you clarify 
what you mean?)

If the Kconfig tree has conflicting symbols in it (dependencies on something 
it can't find), then yeah it would exit with an error in that case.  But I'm 
not sure that's a down side, that's detecting a bug.

My use case is that I'm trying to run the build within a script, and if 
something goes wrong I want the build to abort then rather than have to 
backtrack later.  And when I'm testing it out from the command line, several 
times I've typoed "KCONFIG_ALLCONFIG=../path/to/miniconfig", and right now it 
happily does an allnoconfig that I have to examine to see if it's correct.  
For allnoconfig, there not being an allno.conf file is fine, but for 
miniconfig the action would be meaningless if the mini.conf isn't found, so 
it should error.

My take, anyway.  I'll come up with a patch...

Rob

  reply	other threads:[~2005-11-20  5:54 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-17 12:29 Quick and dirty miniconfig howto, with feature suggestions Rob Landley
2005-11-20  3:08 ` Roman Zippel
2005-11-20  5:54   ` Rob Landley [this message]
2005-11-21  6:15   ` [PATCH] make miniconfig Rob Landley
2005-11-21 16:06     ` [PATCH] make miniconfig (take 2) Rob Landley
2005-11-21 17:26       ` Pavel Machek
2005-11-21 17:46         ` Rob Landley
2005-11-21 17:36       ` Pavel Machek
2005-11-21 17:50         ` Rob Landley
2005-11-21 18:28           ` Pavel Machek
2005-11-21 18:53             ` Rob Landley
2005-11-21 19:24               ` Pavel Machek
2005-11-22  5:14                 ` Rob Landley
2005-11-22 22:54                   ` Pavel Machek
2005-11-23  8:58                     ` Rob Landley
2005-11-23 13:21                       ` Pavel Machek
2005-11-24  3:19                         ` Rob Landley
2005-11-25 19:45                           ` Pavel Machek
2005-11-25 21:45                             ` Rob Landley
2005-11-25 22:09                               ` Pavel Machek
2005-11-25 22:20                                 ` Rob Landley
2005-11-26  2:34                                   ` Kyle Moffett
2005-11-26 12:25                                     ` Rob Landley
2005-11-26 14:19                                       ` Pavel Machek
2005-11-26 15:51                                         ` Rob Landley
2005-11-24  4:02                         ` Rob Landley
2005-11-25 19:46                           ` Pavel Machek
2005-11-26  1:53                             ` Andre Noll
2005-11-22  5:17                 ` Rob Landley
2005-11-24 13:56       ` Roman Zippel
2005-11-24 17:45         ` Rob Landley
2005-11-25  1:48           ` Roman Zippel
2005-11-25  8:18             ` Rob Landley
2005-11-25 15:02               ` Roman Zippel
2005-11-25 16:47                 ` Rob Landley
2005-11-25 21:12             ` Rob Landley
2005-11-27 23:20               ` Roman Zippel
2005-11-28  0:59                 ` Rob Landley
2005-11-29  1:00                   ` Roman Zippel
2005-11-29  2:26                     ` Rob Landley
2005-11-30  2:29                       ` Roman Zippel
2005-11-20  3:25 ` Quick and dirty miniconfig howto, with feature suggestions Rob Landley

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=200511192354.19108.rob@landley.net \
    --to=rob@landley.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.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.