diff mbox series

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

Message ID 20200911015421.22193-1-unixbhaskar@gmail.com
State New
Headers show
Series [V2] scripts : config : Added example use when run without argument,extend help | expand

Commit Message

Bhaskar Chowdhury Sept. 11, 2020, 1:54 a.m. UTC
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(+)

Comments

Randy Dunlap Sept. 11, 2020, 3:08 a.m. UTC | #1
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.
Bhaskar Chowdhury Sept. 11, 2020, 3:40 a.m. UTC | #2
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
>
diff mbox series

Patch

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