checkAUTHORS.pl -> updateAUTHORS.pl in all cases

This removes C<checkAUTHORS.pl> and replaces it in nearly all use cases
with C<updateAUTHORS.pl>.

The only use case I am aware of that is not supported is piping a set of
commit info to a file so it can be parsed separately, and interfacing
with github to look up github accounts. C<updateAUTHORS.pl> wasn't doing
this anyway so it shouldn't matter much.

Virtually every option supported by C<checkAUTHORS.pl> is
supported by C<updateAUTHORS.pl>, along with multiple new
features, with more to come.

This patch updates t/porting/authors.t to use C<updateAUTHORS.pl> for
verifying that the AUTHORS and .mailmap data is up to date. Keeping
checkAUTHORS.pl in sync with how updateAUTHORS.pl works once the
excluded contributors file was created was a pain.

There are new reporting options as well. --files will show the files
modified during the time period, --activity will show the same but
sorted by lines added, --chainsaw will show the same but sorted lines
removed. Using any of the files related options will likely be slow if
you do it on the full history.

As part of this patch the code in the tool has been split in two. That
defining options to the tool stayed where it was, that implementing the
internal logic has been moved to F<Porting/updateAUTHORS.pm>.

This removes one test file F<t/porting/pending-authors.t> as I
couldn't figure out why it was necessary. I am pretty sure it should
not be anymore.

Author: demerphq

MANIFEST

@@ -5386,7 +5386,6 @@ Porting/bisect-runner.pl	Tool to be called by git bisect run
 Porting/bump-perl-version	bump the perl version in relevant files
 Porting/check-cpan-pollution	Check for commits that may wrongly touch CPAN distros
 Porting/checkansi.pl		Check source code for ANSI-C violations
-Porting/checkAUTHORS.pl		Check that the AUTHORS file is complete
 Porting/checkcfguse.pl		Check that config symbols are being used
 Porting/checkcfgvar.pl		Check that config scripts define all symbols
 Porting/checkpodencoding.pl	Check POD encoding
@@ -5452,6 +5451,7 @@ Porting/timecheck.c		Test program for the 2038 fix
 Porting/timecheck2.c		Test program for the 2038 fix
 Porting/todo.pod		Perl things to do
 Porting/updateAUTHORS.pl	Tool to automatically update AUTHORS and .mailmap from git log data
+Porting/updateAUTHORS.pm	Brains of updateAUTHORS.pl
 Porting/valgrindpp.pl		Summarize valgrind reports
 Porting/vote_admin_guide.pod	Perlgov Vote Administrator guide
 pp.c				Push/Pop code
@@ -6073,7 +6073,6 @@ t/porting/known_pod_issues.dat	Data file for porting/podcheck.t
 t/porting/libperl.t		Check libperl.a sanity
 t/porting/maintainers.t		Test that Porting/Maintainers.pl is up to date
 t/porting/manifest.t		Test that this MANIFEST file is well formed
-t/porting/pending-author.t	Check if any pending commit would break tests
 t/porting/perlfunc.t		Test that Functions_pm.PL can parse perlfunc.pod
 t/porting/pod_rules.t		Check that various pod lists are consistent
 t/porting/podcheck.t		Test the POD of shipped modules is well formed

Porting/README.pod

@@ -37,12 +37,6 @@ number in various files within the perl source.
 
 Check source code for ANSI-C violations.
 
-=head2 F<checkAUTHORS.pl>
-
-Used by F<t/porting/authors.t> to ensure the F<AUTHORS> list is up to
-date. See also L<< /"F<updateAUTHORS.pl>" >> for a way to automatically
-fix issues found by this tool.
-
 =head2 F<checkcfguse.pl>
 
 Check where the symbols defined in the various F<config.sh>-clones
@@ -149,7 +143,7 @@ List of Perl release epigraphs.
 =head2 F<exclude_contrib.txt>
 
 List of base 64 encoded SHA256 digests of C<< "name <email>" >> data
-which should be ignored by F<checkAUTHORS.pl> and F<updateAUTHORS.pl>.
+which should be ignored by F<updateAUTHORS.pl>.
 
 =head2 F<exec-bit.txt>
 
@@ -386,10 +380,15 @@ any previous attempts.
 =head2 F<updateAUTHORS.pl>
 
 This script will automatically update AUTHORS and create .mailmap entries
-based on the git commit log history. If F<checkAUTHORS.pl> complains
+based on the git commit log history. If F<t/porting/authors.t> complains
 during testing you should run this. It will automatically fix most if not
 all AUTHORS related test fails.
 
+=head2 F<updateAUTHORS.pm>
+
+The class used by F<updateAUTHORS.pl> to do its business. Package name
+C<Porting::updateAUTHORS>.
+
 =head2 F<valgrindpp.pl>
 
 A post-processor for C<make test.valgrind>.  F<valgrindpp.pl> is a
@@ -403,4 +402,3 @@ leaks.
 Guide for Vote Administrators for running Steering Council elections.
 
 =cut
-

Porting/acknowledgements.pl

@@ -163,5 +163,5 @@ sub commify {
 # returns a list of the authors
 sub authors {
     return
-        qx(git log --pretty=fuller $since_until | $^X Porting/checkAUTHORS.pl --who -);
+        qx($^X Porting/updateAUTHORS.pl --who $since_until);
 }