Branch: refs/heads/smoke-me/khw-asap
  Home:   https://github.com/Perl/perl5
  Commit: 28b41ec3b07e231d99b73da463746a78123879ed
      
https://github.com/Perl/perl5/commit/28b41ec3b07e231d99b73da463746a78123879ed
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  XXX temp debug


  Commit: 2f7d3d22782601e0d79c5eb00613732aa0b7a1c9
      
https://github.com/Perl/perl5/commit/2f7d3d22782601e0d79c5eb00613732aa0b7a1c9
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Extract sort into function

This is in preparation for it being called in other places, in a future
commit.


  Commit: 8c4760801f88dbc28a7676e79b2a800ea2894931
      
https://github.com/Perl/perl5/commit/8c4760801f88dbc28a7676e79b2a800ea2894931
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  There is a .pl test file: cpan/Term-Cap/test.pl

It was handled by using '*' for that directory, but future commits will
want to name individual files in a directory.


  Commit: f447274e05e628d187f5b73bcedc2bf79347b55c
      
https://github.com/Perl/perl5/commit/f447274e05e628d187f5b73bcedc2bf79347b55c
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Assign name to $2

It's clearer, and a future commit will destroy $2 before its final use


  Commit: 390f46a92ab82197d55623e0b10749ee98247890
      
https://github.com/Perl/perl5/commit/390f46a92ab82197d55623e0b10749ee98247890
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Assign names to $_

These are clearer to read, espectially in largish loops.


  Commit: 4ec62e6720ee12e20bcd2d356407c1692bf7df4e
      
https://github.com/Perl/perl5/commit/4ec62e6720ee12e20bcd2d356407c1692bf7df4e
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  ..

diff shows no difference


  Commit: f1888ca403e59bfaececc47665510853e3a4e921
      
https://github.com/Perl/perl5/commit/f1888ca403e59bfaececc47665510853e3a4e921
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Change tabs to spaces


  Commit: 4ec18fd00555163857c908df291a92425e3be9f0
      
https://github.com/Perl/perl5/commit/4ec18fd00555163857c908df291a92425e3be9f0
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  l


  Commit: c05d770c4c9881a2fa4f5b4e6aca388bdf9bca5e
      
https://github.com/Perl/perl5/commit/c05d770c4c9881a2fa4f5b4e6aca388bdf9bca5e
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Move ARGV, ENV processing earlier

This is in preparation for a future commit.

That is the purpose of this commit; however, the git difference listing
instead shows moving the _extract_tests() function later, because that
is the more minimal difference.


  Commit: 7858282f03fe10f2b132b9e190eb4d811cdb5937
      
https://github.com/Perl/perl5/commit/7858282f03fe10f2b132b9e190eb4d811cdb5937
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Improve handling of serial execution exceptions

Experience has shown that some directories must be serially executed
even though they have no rules indicating so.  There are currently 2
such directories, hard-coded as such.  This commit turns things into a
hash, removing a grep and does the check within an already existing
loop.  If an exception is found, it is deleted from the hash, and at the
end, only the non-matched ones remain, so it is trivial to output that
these weren't found, which indicastes a likely error.


  Commit: 905753a71a05981005ab563c2742603b39d3e336
      
https://github.com/Perl/perl5/commit/905753a71a05981005ab563c2742603b39d3e336
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Move comments

This wasn't done in the previsous commit in order to minimize the diff
listing


  Commit: 52cb154c779f1c1564c8534223d507ccd323e6be
      
https://github.com/Perl/perl5/commit/52cb154c779f1c1564c8534223d507ccd323e6be
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Extract code into a function

This is in preparation for it being called from a second place in the
following commit.


  Commit: 5ccc1f97ef557fb5cbdfd3bfe6d76570036b4c44
      
https://github.com/Perl/perl5/commit/5ccc1f97ef557fb5cbdfd3bfe6d76570036b4c44
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: Run more parallel tests by execution time order

In the normal mode of operation of t/harness, there are three sets of
tests, where the first set has to completely finish before the second
one starts, which has to completely finish before the third set starts.

The first set is the basic sanity, and quickly completes.  That is
untouched by this commit.

The third set are the tests for mostly non-core modules, including
cpan/, dist/, ext/, and lib/.  These are executed so that the
longest-running ones are started first when parallel testing is
available.  That is also unchanged by this commit.

The second set is what's left over, generally core tests and ones
enabled only on some platforms or by specifying an option, such as
torture tests.  Prior to this commit, these were executed in parallel,
if permitted, but there was no effort to start the longest-running ones
first.  This results in a less than optimum use of the CPU.

The code that orders tests by running time was previously applied to
just the third set of tests.  Prior commits have extracted out that code
into a function.  This commit just uses that function for the tests
in the second set.

In principle, the basic sanity tests could also be run in parallel.
This would be easy to do, but I'm a little leery, and since they are
quick to execute there would not be much gain in doing so.

Comments remaining in the code indicate that the intent was to execute
ext/ and lib/ tests last.  But that hasn't been true for quite some
time, and no one seems to have noticed.  Now that we have a function to
assemble the tests in a given directory, it would be trivial to change
back, but that does lead to a longer wall clock execution time.

The PERL_TEST_HARNESS_ASAP environment variable is used to change the
mode of operation so that most of the tests in the second set are
instead moved into the third set.  Only the comp/, run/, and cmd tests
are left in the second set.  That is also unchanged by this commit.


  Commit: af326d929c371e8dc2d0aef7de41bc4feb314d9d
      
https://github.com/Perl/perl5/commit/af326d929c371e8dc2d0aef7de41bc4feb314d9d
  Author: Karl Williamson <[email protected]>
  Date:   2022-12-13 (Tue, 13 Dec 2022)

  Changed paths:
    M t/harness

  Log Message:
  -----------
  harness: XXX ASAP with comp/run/cmd


Compare: https://github.com/Perl/perl5/compare/1d90d74f3428...af326d929c37

Reply via email to