Re: [PATCH V2] scripts : config : Added example use when run without argument,extend help

[Bhaskar Chowdhury <unixbhaskar@xxxxxxxxx>]

On 20:08 Thu 10 Sep 2020, Randy Dunlap wrote:
> On 9/10/20 6:54 PM, Bhaskar Chowdhury wrote:
> > This patch extends the help section by adding an explicit example of use.
> >
> > Signed-off-by: Bhaskar Chowdhury <unixbhaskar@xxxxxxxxx>
> > ---
>
> Difference between versions goes here.
>
> >  scripts/config | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> >
> > diff --git a/scripts/config b/scripts/config
> > index eee5b7f3a092..d905dccabb94 100755
> > --- a/scripts/config
> > +++ b/scripts/config
> > @@ -45,6 +45,23 @@ make time.
> >  By default, $myname will upper-case the given symbol. Use --keep-case to keep
> >  the case of all following symbols unchanged.
> >  
> > +The concrete example, say, you want to change any particular config,like
>
>                                                                config, like
>
> > +GCOV for profiling, you can simply use this command
>
>                                               command.
>
> > +
> > +To enable:
> > +
> > +#scripts/config --enable GCOV_KERNEL  && grep GCOV .config
> > +
> > + and the output will be like this:
> > +
> > +  CONFIG_GCOV_KERNEL=y
> > +
> > +To disable:
> > +
> > +#scripts/config --disable GCOV_KERNEL  && grep GCOV .config
> > +
> > +# CONFIG_GCOV_KERNEL is not set
> > +
> >  $myname uses 'CONFIG_' as the default symbol prefix. Set the environment
> >  variable CONFIG_ to the prefix to use. Eg.: CONFIG_="FOO_" $myname ...
> >  EOL
>
> I don't know why we need this extra help text.

I am not sure I get you ...should have mentioned in the first place, so it
save some of my time ....moreover, I was doing it because I felt it necessary
to let the user of this script to know what/how easily things can be
done.Period.

Look like decision making is badly bugged. 

~Bhaskar 
> --
> ~Randy

Attachment: signature.asc
Description: PGP signature