Re: [PATCH] get_maintainer: don't remind about no git repo when --nogit is used

[Joe Perches]

On Sat, 2022-01-01 at 19:14 -0800, Randy Dunlap wrote:
> When --nogit is used with scripts/get_maintainer.pl, the script
> spews 4 lines of unnecessary information (noise). Do not print those
> lines when --nogit is specified.

Seems sensible, thanks.

> 
> This change removes the printing of these 4 lines:
> 
>   ./scripts/get_maintainer.pl: No supported VCS found.  Add --nogit to options?
>   Using a git repository produces better results.
>   Try Linus Torvalds' latest git repository using:
>   git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> 
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Joe Perches <joe@xxxxxxxxxxx>
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
>  scripts/get_maintainer.pl |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-next-20211224.orig/scripts/get_maintainer.pl
> +++ linux-next-20211224/scripts/get_maintainer.pl
> @@ -1718,7 +1718,7 @@ sub vcs_exists {
>      %VCS_cmds = %VCS_cmds_hg;
>      return 2 if eval $VCS_cmds{"available"};
>      %VCS_cmds = ();
> -    if (!$printed_novcs) {
> +    if (!$printed_novcs && $email_git) {
>  	warn("$P: No supported VCS found.  Add --nogit to options?\n");
>  	warn("Using a git repository produces better results.\n");
>  	warn("Try Linus Torvalds' latest git repository using:\n");