linux-embedded.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tim Bird <tim.bird@am.sony.com>
To: David Woodhouse <dwmw2@infradead.org>
Cc: linux-embedded <linux-embedded@vger.kernel.org>,
	linux kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] add diffconfig utility
Date: Tue, 10 Jun 2008 15:33:06 -0700	[thread overview]
Message-ID: <484F0122.6080700@am.sony.com> (raw)
In-Reply-To: <1213135624.2534.99.camel@shinybook.infradead.org>

David Woodhouse wrote:
> On Tue, 2008-06-10 at 12:41 -0700, Tim Bird wrote:
>> +# Added and removed items are shown with a leading plus or minus,
>> +# respectively.  Changed items show the old and new values on a
>> +# single line.
> 
> It'd be really nice if it could give its output in the same form as
> the .config file itself -- so it'd look something like:
> 
> # CONFIG_FOO is not set
> CONFIG_BAR=y

Would an option for that style of output be OK?

The tool currently produces something similar
to a diff (but with changed lines compressed to
a single line for conciseness).  It should be easy
to produce something that can be used with another
tool to transform one config into another.

It's unclear what to do with variables that are
supposed to be removed from the config (as opposed
to be set to "CONFIG_FOO is not set"  Does merge.pl
handle this, or is this left to something like
'make oldconfig'?
 -- Tim

=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================


  reply	other threads:[~2008-06-10 22:33 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-10 19:41 [PATCH] add diffconfig utility Tim Bird
2008-06-10 21:02 ` Jörn Engel
2008-06-10 21:18   ` Sam Ravnborg
2008-06-10 21:33     ` execute bit on scripts (was Re: [PATCH] add diffconfig utility) Tim Bird
2008-06-10 22:26       ` Jan Engelhardt
2008-06-10 21:21 ` [PATCH] add diffconfig utility Sam Ravnborg
2008-06-11  0:25   ` Tim Bird
2008-06-11 11:28     ` Geert Uytterhoeven
2008-06-11 18:23       ` Tim Bird
2008-06-12 13:19         ` Sam Ravnborg
2008-06-12 16:07           ` Tim Bird
2008-06-24 17:56           ` [PATCH] add diffconfig utility (v2) Tim Bird
2008-06-27 21:10             ` Sam Ravnborg
2008-06-10 22:07 ` [PATCH] add diffconfig utility David Woodhouse
2008-06-10 22:33   ` Tim Bird [this message]
2008-06-10 23:02   ` Arjan van de Ven
2008-06-10 23:10     ` David Woodhouse
2008-06-11  6:59 ` Holger Schurig

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=484F0122.6080700@am.sony.com \
    --to=tim.bird@am.sony.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-embedded@vger.kernel.org \
    --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 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).