develooper Front page | perl.perl5.changes | Postings from August 2022

[Perl/perl5] b12cb3: add missing .mailmap entries

From:
Yves Orton via perl5-changes
Date:
August 8, 2022 19:20
Subject:
[Perl/perl5] b12cb3: add missing .mailmap entries
Message ID:
Perl/perl5/push/refs/heads/yves/exclude_contrib/000000-fc2323@github.com
  Branch: refs/heads/yves/exclude_contrib
  Home:   https://github.com/Perl/perl5
  Commit: b12cb3b3351f5a81e32c1dcfca965ea51fa21b49
      https://github.com/Perl/perl5/commit/b12cb3b3351f5a81e32c1dcfca965ea51fa21b49
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-07 (Sun, 07 Aug 2022)

  Changed paths:
    M .mailmap

  Log Message:
  -----------
  add missing .mailmap entries

Current Porting/updateAUTHORS.pl and t/porting/authors.t have slightly different ideas
of when data is missing.


  Commit: 905a69fe65ea7b3e90c7d09f989ba093bd665e15
      https://github.com/Perl/perl5/commit/905a69fe65ea7b3e90c7d09f989ba093bd665e15
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-07 (Sun, 07 Aug 2022)

  Changed paths:
    M Porting/updateAUTHORS.pl

  Log Message:
  -----------
  Porting/updateAUTHORS.pl - refactor to be OO

Makes managing the state we need much easier.


  Commit: 5d361f8922506015e55587da25fd40fadfd40d03
      https://github.com/Perl/perl5/commit/5d361f8922506015e55587da25fd40fadfd40d03
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-07 (Sun, 07 Aug 2022)

  Changed paths:
    M .mailmap
    M AUTHORS
    M MANIFEST
    M Porting/README.pod
    A Porting/exclude_contrib.txt
    M Porting/updateAUTHORS.pl
    M pod/perl5340delta.pod
    M t/porting/authors.t

  Log Message:
  -----------
  Porting/updateAUTHORS.pl - add support for excluding contributors

This adds a new file Porting/exclude_contrib.pl which is managed by
Porting/updateAUTHORS.pl to exclude contributors by the base64 SHA-256
digest of their name.

This also uses this new feature to remove someone at their request from
our files.

This also changes t/porting/authors.t to no longer use
Porting/checkAUTHORS.pl which does not do the right thing these days
anyway. Instead Porting/updateAUTHORS.pl now knows how to check the
files and emit TAP.


  Commit: 8519327680c5d7248b40179844fac38ec547694a
      https://github.com/Perl/perl5/commit/8519327680c5d7248b40179844fac38ec547694a
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-08 (Mon, 08 Aug 2022)

  Changed paths:
    M AUTHORS

  Log Message:
  -----------
  Improved header comment in AUTHORS file

At the very least the file is no longer updated or managed
with checkAUTHORS.pl


  Commit: fc2323daa1bf4fe47a19c78f66c22fa5c84be717
      https://github.com/Perl/perl5/commit/fc2323daa1bf4fe47a19c78f66c22fa5c84be717
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-08 (Mon, 08 Aug 2022)

  Changed paths:
    M MANIFEST
    M Porting/README.pod
    M Porting/acknowledgements.pl
    R Porting/checkAUTHORS.pl
    M Porting/exec-bit.txt
    M Porting/release_managers_guide.pod
    M Porting/updateAUTHORS.pl
    A Porting/updateAUTHORS.pm
    M README.cygwin
    R t/porting/pending-author.t

  Log Message:
  -----------
  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.


Compare: https://github.com/Perl/perl5/compare/b12cb3b3351f%5E...fc2323daa1bf



nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About