From: Craig A. Berry <craigberry@mac.com>
Date: Sat, 27 Mar 2010 17:52:33 +0000 (-0400)
Subject: Update Porting/checkAUTHORS.pl to (once again) distinguish authors from committers... 
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=94e9284742fce739393f3ccf1f443b4486b689cc;p=p5sagit%2Fp5-mst-13.2.git

Update Porting/checkAUTHORS.pl to (once again) distinguish authors from committers by processing git log --pretty=fuller, e.g.,

% git log --pretty=fuller perl-5.12.0..HEAD \
? | perl Porting/checkAUTHORS.pl --rank --percentage -

Without this, the --rank and --thanks-applied options were not working.

Also, the meaning of --thanks-applied has changed somewhat; it now only counts
commits where the committer is not the author.

[This commit is the delta between Craig's previous patch and his revised
patch]
---

diff --git a/Porting/checkAUTHORS.pl b/Porting/checkAUTHORS.pl
index 4620c68..4b1309e 100644
--- a/Porting/checkAUTHORS.pl
+++ b/Porting/checkAUTHORS.pl
@@ -24,7 +24,7 @@ $0 --percentage ...                         # show rankings as percentages
 $0 --cumulative ...                         # show rankings cumulatively
 $0 --reverse ...                            # show rankings in reverse
 Specify stdin as - if needs be. Remember that option names can be abbreviated.
-Generate changes with git log --pretty=fuller --name-status rev1..rev2
+Generate changes with git log --pretty=fuller rev1..rev2
 EOS
 }
 
diff --git a/Porting/release_managers_guide.pod b/Porting/release_managers_guide.pod
index c665547..ffd02a4 100644
--- a/Porting/release_managers_guide.pod
+++ b/Porting/release_managers_guide.pod
@@ -365,7 +365,7 @@ Update F<AUTHORS>, using the C<Porting/checkAUTHORS.pl> script, and if
 necessary, update the script to include new alias mappings for porters
 already in F<AUTHORS>
 
-	$ git log | perl Porting/checkAUTHORS.pl --acknowledged AUTHORS -
+    $ git log --pretty=fuller | perl Porting/checkAUTHORS.pl --acknowledged AUTHORS -
 
 =back