Re: [PATCH 2/2] [RFC] kconfig/listnewconfig: show default value of new symbel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2010-12-21 at 17:47 +0100, Michal Marek wrote:
> On 5.12.2010 07:35, Arnaud Lacombe wrote:
> > note: I would not mind adding the KCONFIG_VERBOSE logic.
> > 
> > Original idea and implementation by Bastian Blank <waldi@xxxxxxxxxx>,
> > Maximilian Attems <max@xxxxxxx> and Ben Hutchings <ben@xxxxxxxxxxxxxxx>
> > 
> > CC: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
> > Signed-off-by: Arnaud Lacombe <lacombar@xxxxxxxxx>
> > ---
> >  scripts/kconfig/conf.c     |   14 +++++++++++++-
> >  scripts/kconfig/confdata.c |    4 ++--
> >  2 files changed, 15 insertions(+), 3 deletions(-)
> > 
> > diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c
> > index 5459a38..1b58129 100644
> > --- a/scripts/kconfig/conf.c
> > +++ b/scripts/kconfig/conf.c
> > @@ -411,6 +411,17 @@ conf_childs:
> >  		indent -= 2;
> >  }
> >  
> > +
> > +static void
> > +listnewconfig_print_symbol(FILE *fp, struct symbol *sym, const char *val, void *arg)
> > +{
> > +	fprintf(fp, "%s%s (default: %s)\n", CONFIG_, sym->name, val);
> > +}
> 
> Ben, do you have an opinion on this format versus the standard .config
> produced by your patch?
[...]

The .config format is more useful as it is then possible to copy lines
directly into a .config file.  If you're going to apply Arnaud's
preceding patch (1/2) then please let me rewrite mine to apply on top of
that, rather than applying this second patch.

Ben.

-- 
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux