Re: Wrong PR e-mails

[Thomas Klausner <tk%giga.or.at@localhost>]

On Wed, May 13, 2015 at 02:16:00PM +0000, Christos Zoulas wrote:
> No, I feel sorry for you. Perhaps we should modify the script so that
> if it finds more than 2 PR matches it makes the commit fail, or it does
> not add anymore...

I suggest the attached patch (with "more than 3" PRs, but you can
easily change it to "more than 2").
 Thomas

Index: log_accum
===================================================================
RCS file: /cvsroot/CVSROOT/log_accum,v
retrieving revision 1.112
diff -u -r1.112 log_accum
--- log_accum	21 Oct 2014 13:21:08 -0000	1.112
+++ log_accum	14 May 2015 17:52:23 -0000
@@ -433,6 +433,8 @@
     @prs = map { if (!$last || $last ne $_) { $last = $_ ; } else { () } } sort @prs;
     # discard PR numbers below 10000 because they are probably accidents like github pull-requests
     @prs = grep { $_ > 10000 } @prs;
+    # if more than 3 PRs are cited, assume it's an accident like a pkgsrc changelog
+    @prs = () if (@prs > 3);
 
     return @prs;
 }