Paolo Bonzini <[email protected]> writes:
> This can be useful, at the very least, for debugging.
>
> Signed-off-by: Paolo Bonzini <[email protected]>
> ---
> scripts/get_maintainer.pl | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 38334de..a3a16d8 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -55,7 +55,7 @@ my $help = 0;
>
> my $vcs_used = 0;
>
> -my $exit = 0;
> +my $exit = 1;
>
> my %commit_author_hash;
> my %commit_signer_hash;
> @@ -2061,6 +2061,7 @@ sub output {
> print(join($output_separator, @parms));
> print("\n");
> }
> + $exit = 0 if @parms > 0;
> }
>
> my $rfc822re;
Not sure about "if no maintainer found".
Here's how output() gets called:
if (@maintainers) {
@maintainers = merge_email(@maintainers);
output(@maintainers);
}
if ($scm) {
@scm = uniq(@scm);
output(@scm);
}
if ($status) {
@status = uniq(@status);
output(@status);
}
if ($subsystem) {
@subsystem = uniq(@subsystem);
output(@subsystem);
}
if ($web) {
@web = uniq(@web);
output(@web);
}
exit($exit);
Thus, the patch makes the script exit with status 1 when none of the
above output() calls produces output.
Maybe it exits with status 1 when it fails to produce output? Obvious
ouput elsewhere: printing the version (okay), and a print in
get_maintainers(). Can't tell what the latter does. Hmm.
