Messages in this thread Patch in this message | | | From | florian@mickler ... | Subject | [PATCH] get_maintainer.pl: ignore non-maintainer tags | Date | Fri, 7 May 2010 08:34:55 +0200 |
| |
Using --git to determine who to send a patch to, it is not reasonable to include people that only reported an issue or tested a patch. Thus we restrict the candidates to be the one's listed with Reviewed-By, Signed-Off-By and Acked-By tags.
The Acked-By: is questionable also, but as people listed with this tag tend to be active linux gatekeepers, false positives are tolerable.
Signed-off-by: Florian Mickler <florian@mickler.org> --- scripts/get_maintainer.pl | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl index 6f97a13..ad38ed3 100755 --- a/scripts/get_maintainer.pl +++ b/scripts/get_maintainer.pl @@ -496,7 +496,7 @@ version: $V MAINTAINER field selection options: --email => print email address(es) if any - --git => include recent git \*-by: signers + --git => include recent git {Reviewed|Signed-Off|Acked}-by: signers --git-chief-penguins => include ${penguin_chiefs} --git-min-signatures => number of signatures required (default: 1) --git-max-maintainers => maximum maintainers to add (default: 5) @@ -964,7 +964,7 @@ sub vcs_find_signers { $commits = grep(/$pattern/, @lines); # of commits - @lines = grep(/^[-_ a-z]+by:.*\@.*$/i, @lines); + @lines = grep(/(signed-off|reviewed|acked)-by:.*\@.*$/i, @lines); if (!$email_git_penguin_chiefs) { @lines = grep(!/${penguin_chiefs}/i, @lines); } -- 1.7.0.4
| |