This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Gisle points out that it's ok to ignore the return value of newSVrv.
[perl5.git] / Porting / checkAUTHORS.pl
index 798dd0c..4468667 100644 (file)
@@ -5,14 +5,16 @@ $Text::Wrap::columns = 80;
 my ($committer, $patch, $log);
 use Getopt::Long;
 
-my ($rank, @authors, %authors, %untraced, %patchers);
-my $result = GetOptions ("rank" => \$rank,                     # rank authors
-                        "acknowledged=s"   => \@authors);      # authors files
+my ($rank, $ta, @authors, %authors, %untraced, %patchers, %committers);
+my $result = GetOptions ("rank" => \$rank,                 # rank authors
+                        "thanks-applied" => \$ta,          # ranks committers
+                        "acknowledged=s"   => \@authors);  # authors files
 
-if (!$result or !($rank xor @authors) or !@ARGV) {
+if (!$result or (($rank||0) + ($ta||0) + (@authors ? 1 : 0) != 1) or !@ARGV) {
   die <<"EOS";
 $0 --rank Changelogs                        # rank authors by patches
 $0 --acknowledged <authors file> Changelogs # Display unacknowledged authors
+$0 --thanks-applied Changelogs             # ranks committers
 Specify stdin as - if needs be. Remember that option names can be abbreviated.
 EOS
 }
@@ -86,33 +88,38 @@ my %map = reverse (
                   "thomas.dorner\100start.de" => "tdorner\100amadeus.net",
                   "ajohnson\100nvidia.com" => "ajohnson\100wischip.com",
                   "phil\100perkpartners.com" => "phil\100finchcomputer.com",
+                  "tom.horsley\100mail.ccur.com" => "tom.horsley\100ccur.com",
+                  "rootbeer\100teleport.com" => "rootbeer\100redcat.com",
+                  "cp\100onsitetech.com" => "publiustemp-p5p\100yahoo.com",
+                  "epeschko\100den-mdev1" => "esp5\100pge.com",
+                  "pimlott\100idiomtech.com" => "andrew\100pimlott.net",
+                  "fugazi\100zyx.net" => "larrysh\100cpan.org",
+                  "merijnb\100iloquent.nl" => "merijnb\100iloquent.com",
+                  "whatever\100davidnicol.com" => "davidnicol\100gmail.com",
+                  "rmgiroux\100acm.org" => "rmgiroux\100hotmail.com",
+                  "smcc\100mit.edu" => "smcc\100ocf.berkeley.edu",
+                  "steven\100accognoscere.org" => "schubiger\100cpan.org",
+                  "richard.foley\100ubsw.com"
+                  => "richard.foley\100t-online.de",
+                  "damian\100cs.monash.edu.au" => "damian\100conway.org",
+                  "gp\100familiehaase.de" => "gerrit\100familiehaase.de",
+                  "juerd\100cpan.org" => "juerd\100convolution.nl",
+                  "paul.green\100stratus.com"
+                  => "paul_greenvos\100vos.stratus.com",
+                  "alian\100cpan.org" => "alian\100alianwebserver.com",
                   # Maybe we should special case this to get real names out?
                   "perlbug\100perl.org" => "perlbug-followup\100perl.org",
                  );
 
 # Make sure these are all lower case.
 
-$map{"alan.burlison\100uk.sun.com"} = "alanbur";
-$map{"artur\100contiller.se"} = $map{"arthur\100contiller.se"} = "sky";
 $map{"autrijus\100egb.elixus.org"} = $map{"autrijus\100geb.elixus.org"}
+  = $map{"autrijus\100gmail.com"} = $map{"autrijus\100ossf.iis.sinica.edu.tw"}
   = "autrijus\100autrijus.org";
-$map{"craig.berry\100psinetcs.com"} = $map{"craig.berry\100metamorgs.com"}
-  = $map{"craig.berry\100signaltreesolutions.com"}
-  = $map{"craigberry\100mac.com"} = "craigb";
-$map{"davem\100iabyn.nospamdeletethisbit.com" }
-  = $map{"davem\100fdgroup.co.uk"} = "davem";
 $map{"ilya\100math.ohio-state.edu"} = $map{"ilya\100math.berkeley.edu"}
   = $map{"ilya\100math.berkeley.edu"} = "nospam-abuse\100ilyaz.org";
-$map{"jhietaniemi\100gmail.com"} = $map{"jhi\100kosh.hut.fi"}
-  = $map{"jhi\100cc.hut.fi"} = "jhi";
-$map{"nick\100ccl4.org"} = $map{"nick\100talking.bollo.cx"}
-  = $map{"nick\100plum.flirble.org"} = $map{"nick\100babyhippo.co.uk"}
-  = $map{"nick\100bagpuss.unfortu.net"} = "nicholas";
 $map{"philip.newton\100gmx.net"} = $map{"philip.newton\100datenrevision.de"}
   = $map{"pnewton\100gmx.de"} = "pne\100cpan.org",
-$map{"rgarciasuarez\100mandrakesoft.com"}
-  = $map{"rgarciasuarez\100mandriva.com"}
-  = $map{"raphel.garcia-suarez\100hexaflux.com"} = "rgs";
 $map{"simon\100pembro4.pmb.ox.ac.uk"} = $map{"simon\100brecon.co.uk"}
   = $map{"simon\100othersideofthe.earth.li"} = $map{"simon\100cozens.net"}
   = $map{"simon\100netthink.co.uk"} = "simon\100simon-cozens.org";
@@ -120,15 +127,11 @@ $map{"spider\100web.zk3.dec.com"} = $map{"spider\100leggy.zk3.dec.com"}
   = $map{"spider-perl\100orb.nashua.nh.us"}
   = $map{"spider\100peano.zk3.dec.com"}
   = "spider\100orb.nashua.nh.us";
-$map{"nik\100tiuk.ti.com"} = "nick";
-
-$map{"a.koenig\100mind.de"} = "andreas.koenig\100anima.de";
+$map{"andreas.koenig.gmwojprw\100franz.ak.mind.de"}
+  = $map{"a.koenig\100mind.de"} =  "andreas.koenig\100anima.de";
 $map{"japhy\100perlmonk.org"} = $map{"japhy\100cpan.org"}
   = "japhy\100pobox.com";
 $map{"rmbarker\100cpan.org"} = "robin.barker\100npl.co.uk";
-
-$map{"h.m.brand\100hccnet.nl"} = $map{"merijn\100l1.procura.nl"} = "merijn";
-
 $map{"yves.orton\100de.mci.com"} = $map{"yves.orton\100mciworldcom.de"}
   = "demerphq\100gmail.com";
 $map{"jim.cromie\100gmail.com"} = "jcromie\100divsol.com";
@@ -140,7 +143,50 @@ $map{"kane\100cpan.org"} = "kane\100dwim.org";
 $map{"rs\100crystalflame.net"} = "p5-authors\100crystalflame.net";
 $map{"(srezic\100iconmobile.com)"} = "slaven\100rezic.de";
 $map{"perl\100dellah.anu.edu.au"} = "spoon\100cpan.org";
+$map{"rjk-perl-p5p\100tamias.net"} = "rjk\100linguist.dartmouth.edu";
+$map{"sts\100accognoscere.org"} = "steven\100accognoscere.org";
+$map{"s.payrard\100wanadoo.fr"} = "stef\100mongueurs.net";
+$map{"richard.foley\100ubs.com"} = "richard.foley\100ubsw.com";
+# I assume that Ton Hopsel's lack of e-mail address in AUTHORS is deliberate
+$map{"me-02\100ton.iguana.be"} = $map{"perl-5.8.0\100ton.iguana.be"}
+  = $map{"perl5-porters\100ton.iguana.be"} = "!";
+# No real name for these address
+$map{$_} = "?" foreach ("grommel\100sears.com", "pxm\100nubz.org",
+                       "padre\100elte.hu", "jdhedden\100" . "1979.usna.com",
+                       "nothingmuch\100woobling.org", "bob\100starlabs.net",
+                       "bbucklan\100jpl-devvax.jpl.nasa.gov",
+                       "bilbo\100ua.fm", "mats\100sm5sxl.net",
+                       "chris\100ex-parrot.com", 
+                       "kaminsky\100math.huji.ac.il",
+                       "bonefish\100cs.tu-berlin.de",
+                       "bstrand\100switchmanagement.com",
+                       "glasser\100tang-eleven-seventy-nine.mit.edu",
+                       "raf\100tradingpost.com.au", "erik\100cs.uni-jena.de",
+                       "jms\100mathras.comcast.net", "kvr\100centrum.cz",
+                       "perlbug\100veggiechinese.net",
+                       "scott\100perlcode.org",
+                      );
+# We don't have an e-mail address for Beau Cox
+$map{"beau\100beaucox.com"} = "?";
 
+$map{"rgarciasuarez\100mandrakesoft.com"}
+  = $map{"rgarciasuarez\100mandriva.com"}
+  = $map{"raphel.garcia-suarez\100hexaflux.com"} = "rgs";
+$map{"jhietaniemi\100gmail.com"} = $map{"jhi\100kosh.hut.fi"}
+  = $map{"jhi\100cc.hut.fi"} = $map{"jarkko.hietaniemi\100nokia.com"} = "jhi";
+$map{"nick\100ccl4.org"} = $map{"nick\100talking.bollo.cx"}
+  = $map{"nick\100plum.flirble.org"} = $map{"nick\100babyhippo.co.uk"}
+  = $map{"nick\100bagpuss.unfortu.net"} = "nicholas";
+$map{"craig.berry\100psinetcs.com"} = $map{"craig.berry\100metamorgs.com"}
+  = $map{"craig.berry\100signaltreesolutions.com"}
+  = $map{"craigberry\100mac.com"} = "craigb";
+$map{"davem\100iabyn.nospamdeletethisbit.com" }
+  = $map{"davem\100fdgroup.co.uk"} = $map{"davem\100fdisolutions.com"}
+ = "davem";
+$map{"alan.burlison\100uk.sun.com"} = "alanbur";
+$map{"artur\100contiller.se"} = $map{"arthur\100contiller.se"} = "sky";
+$map{"h.m.brand\100hccnet.nl"} = $map{"merijn\100l1.procura.nl"} = "merijn";
+$map{"nik\100tiuk.ti.com"} = $map{"nick.ing-simmons\100elixent.com"} = "nick";
 $map{"hv\100crypt.org"} = "hv";
 $map{"gisle\100aas.no"} = "gisle";
 
@@ -168,6 +214,8 @@ if (@authors) {
     $_ = lc $_;
     $authors{$map{$_} || $_}++;
   }
+  ++$authors{'!'};
+  ++$authors{'?'};
 }
 
 while (<>) {
@@ -199,7 +247,9 @@ while (<>) {
 &process ($committer, $patch, $log);
 
 if ($rank) {
-  &display_ordered;
+  &display_ordered(\%patchers);
+} elsif ($ta) {
+  &display_ordered(\%committers);
 } elsif (%authors) {
   my %missing;
   foreach (sort keys %patchers) {
@@ -216,8 +266,9 @@ if ($rank) {
 }
 
 sub display_ordered {
+  my $what = shift;
   my @sorted;
-  while (my ($name, $count) = each %patchers) {
+  while (my ($name, $count) = each %$what) {
     push @{$sorted[$count]}, $name;
   }
 
@@ -232,7 +283,7 @@ sub display_ordered {
 sub process {
   my ($committer, $patch, $log) = @_;
   return unless $committer;
-  my @authors = $log =~ /From:.+\s+([^\@ \t\n]+\@[^\@ \t\n]+)/gm;
+  my @authors = $log =~ /From:\s+.*?([^"\@ \t\n<>]+\@[^"\@ \t\n<>]+)/gm;
 
   if (@authors) {
     foreach (@authors) {
@@ -242,6 +293,7 @@ sub process {
       $patchers{$map{$_} || $_}++;
     }
     # print "$patch: @authors\n";
+    ++$committers{$committer};
   } else {
     # print "$patch: $committer\n";
     # Not entirely fair as this means that the maint pumpking scores for