From: Sam Ravnborg <sam@ravnborg.org>
To: Jens Axboe <axboe@suse.de>, Alan Cox <alan@lxorguk.ukuu.org.uk>,
Arjan van de Ven <arjanv@redhat.com>,
Jeff Garzik <jgarzik@pobox.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: 2.5 vi .config ; make oldconfig not working
Date: Tue, 5 Nov 2002 21:51:01 +0100 [thread overview]
Message-ID: <20021105205101.GA3014@mars.ravnborg.org> (raw)
In-Reply-To: <20021105194229.GS2502@pasky.ji.cz>
On Tue, Nov 05, 2002 at 08:42:29PM +0100, Petr Baudis wrote:
> > The following patch should make most garbage, such as =n, result in
> > # CONFIG_FOO is not set
> > without any user confirmation.
>
> I don't think this is actually a good idea. We break forward compatibility with
> this (possibly, in future we will want to add something like "yes" or you don't
> know what..), and then you will still get "no" for no obvious reason - asking
> is much saner approach here, IMHO. If we didn't understand it, assuming 'no' is
> not a safe way, I believe.
What actually happens is that any non-recognized value is assumed to be 'n'.
So if 'yes' is recognised, then fine.
And with the simple parser used today "CONFIG_FOO=yes" would be recognised
as yes, because it checks only the 'y'.
I did not look that close on the patch you sent, but I think they
have the same functionality.
If this wasn't after feature freeze I would have liked to see the
hidden .config file to be replaced by for example Kconfigdata or
similar.
So many times I have seen people being asked to backup there
configuration, which is located in a hidden file.
If people should keep their fingers away from it - then we should note
that in the top of the file. As we do in the rest of the kernel.
And then we could have invented a better syntax, that was not
dictated by bash or similar.
Something to propose in the 2.7 timeframe.
Sam
next prev parent reply other threads:[~2002-11-05 20:44 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-11-05 16:50 2.5 vi .config ; make oldconfig not working Jens Axboe
2002-11-05 17:08 ` Jeff Garzik
2002-11-05 17:13 ` Dave Jones
2002-11-05 17:16 ` Jens Axboe
2002-11-05 17:21 ` Jeff Garzik
2002-11-05 17:14 ` Jens Axboe
2002-11-05 17:19 ` Jeff Garzik
2002-11-05 17:21 ` Jens Axboe
2002-11-05 18:14 ` Joel Becker
2002-11-05 18:15 ` Jens Axboe
2002-11-06 14:06 ` Roman Zippel
2002-11-06 14:13 ` Jens Axboe
2002-11-05 17:23 ` Randy.Dunlap
2002-11-05 17:26 ` Arjan van de Ven
2002-11-05 17:26 ` Jens Axboe
2002-11-05 17:27 ` Jens Axboe
2002-11-05 18:20 ` Alan Cox
2002-11-05 18:14 ` Jens Axboe
2002-11-05 19:30 ` Sam Ravnborg
2002-11-05 19:42 ` Petr Baudis
2002-11-05 20:51 ` Sam Ravnborg [this message]
2002-11-05 20:16 ` Peter H. Ruegg
2002-11-05 18:55 ` [PATCH] Allow 'n' as a symbol value in the .config file Petr Baudis
2002-11-05 18:59 ` Jens Axboe
2002-11-06 19:39 ` 2.5 vi .config ; make oldconfig not working 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=20021105205101.GA3014@mars.ravnborg.org \
--to=sam@ravnborg.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=arjanv@redhat.com \
--cc=axboe@suse.de \
--cc=jgarzik@pobox.com \
--cc=linux-kernel@vger.kernel.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.