linux-kernel - Re: [PATCH 2/3] kconfig: Wrap long "make help" text lines

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Sat, 30 May 2015 22:10:54 -0300
From:	Diego Viola <diego.viola@...il.com>
To:	Geert Uytterhoeven <geert+renesas@...der.be>
Cc:	Michal Marek <mmarek@...e.cz>,
	"Yann E. MORIN" <yann.morin.1998@...e.fr>,
	linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/3] kconfig: Wrap long "make help" text lines

Hello,

Can you please include a minor change in your patch?

At line 89, there's this:

# on its behavior(sets new symbols to their default value but not 'n') with the

Please change this line to this:

# on its behavior (sets new symbols to their default value but not 'n') with the

There should be a space between "behavior" and the parentheses/bracket.

Thanks,

Diego

On Tue, May 26, 2015 at 8:39 AM, Geert Uytterhoeven
<geert+renesas@...der.be> wrote:
> Some "make help" text lines extend beyond 80 characters. Wrap them at 79
> characters.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>
> ---
>  scripts/kconfig/Makefile | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
> index d9b1fef0c67e3cb4..d19de3665bfbe984 100644
> --- a/scripts/kconfig/Makefile
> +++ b/scripts/kconfig/Makefile
> @@ -122,7 +122,8 @@ tinyconfig:
>  # Help text used by make help
>  help:
>         @echo  '  config          - Update current config utilising a line-oriented program'
> -       @echo  '  nconfig         - Update current config utilising a ncurses menu based program'
> +       @echo  '  nconfig         - Update current config utilising a ncurses menu based'
> +       @echo  '                    program'
>         @echo  '  menuconfig      - Update current config utilising a menu based program'
>         @echo  '  xconfig         - Update current config utilising a QT based front-end'
>         @echo  '  gconfig         - Update current config utilising a GTK based front-end'
> @@ -138,7 +139,8 @@ help:
>         @echo  '  alldefconfig    - New config with all symbols set to default'
>         @echo  '  randconfig      - New config with random answer to all options'
>         @echo  '  listnewconfig   - List new options'
> -       @echo  '  olddefconfig    - Same as silentoldconfig but sets new symbols to their default value'
> +       @echo  '  olddefconfig    - Same as silentoldconfig but sets new symbols to their'
> +       @echo  '                    default value'
>         @echo  '  kvmconfig       - Enable additional options for guest kernel support'
>         @echo  '  tinyconfig      - Configure the tiniest possible kernel'
>
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/