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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20200911034037.GA4063@debian>
Date:   Fri, 11 Sep 2020 09:10:41 +0530
From:   Bhaskar Chowdhury <unixbhaskar@...il.com>
To:     Randy Dunlap <rdunlap@...radead.org>
Cc:     masahiroy@...nel.org, jeremie.francois@...il.com,
        linux-kernel@...r.kernel.org, linux-kbuild@...r.kernel.org
Subject: Re: [PATCH V2] scripts : config : Added example use when run without
 argument,extend help

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@...il.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
>

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)