linux-kbuild.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH V2] scripts : config : Added example use when run without argument,extend help
@ 2020-09-11  1:54 Bhaskar Chowdhury
  2020-09-11  3:08 ` Randy Dunlap
  0 siblings, 1 reply; 3+ messages in thread
From: Bhaskar Chowdhury @ 2020-09-11  1:54 UTC (permalink / raw)
  To: masahiroy
  Cc: jeremie.francois, rdunlap, linux-kernel, linux-kbuild, Bhaskar Chowdhury

This patch extends the help section by adding an explicit example of use.

Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
---
 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
+GCOV for profiling, you can simply use this 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
-- 
2.26.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH V2] scripts : config : Added example use when run without argument,extend help
  2020-09-11  1:54 [PATCH V2] scripts : config : Added example use when run without argument,extend help Bhaskar Chowdhury
@ 2020-09-11  3:08 ` Randy Dunlap
  2020-09-11  3:40   ` Bhaskar Chowdhury
  0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2020-09-11  3:08 UTC (permalink / raw)
  To: Bhaskar Chowdhury, masahiroy; +Cc: jeremie.francois, linux-kernel, linux-kbuild

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@gmail.com>
> ---

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.

-- 
~Randy


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH V2] scripts : config : Added example use when run without argument,extend help
  2020-09-11  3:08 ` Randy Dunlap
@ 2020-09-11  3:40   ` Bhaskar Chowdhury
  0 siblings, 0 replies; 3+ messages in thread
From: Bhaskar Chowdhury @ 2020-09-11  3:40 UTC (permalink / raw)
  To: Randy Dunlap; +Cc: masahiroy, jeremie.francois, linux-kernel, linux-kbuild

[-- Attachment #1: Type: text/plain, Size: 1833 bytes --]

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@gmail.com>
>> ---
>
>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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-09-11  3:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-11  1:54 [PATCH V2] scripts : config : Added example use when run without argument,extend help Bhaskar Chowdhury
2020-09-11  3:08 ` Randy Dunlap
2020-09-11  3:40   ` Bhaskar Chowdhury

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).