Re: [Automake-ng] [PATCH 00/10] Rewrite parallel-tests to rely on GNU make capabilities

[Stefano Lattarini]

On 04/23/2012 11:57 AM, Stefano Lattarini wrote:
> A patch series that rewrites step-by-step a substantial part of the parallel
> testsuite harness, to make it take advantage of more GNU make features.
> It accomplishes this by moving some of the processing from automake runtime
> to make runtime.  And in doing so, it also offers a good example of the
> general improvements and enhanced semantics Automake-NG can offer.
> 
> The sixth and seventh patches are not part of the series proper, but start
> making use of the new features the rewrite offers in Automake's own
> testsuite, and thus also serve as a useful showcase of the advantages these
> features can bring.
> 
> The last two patches fixes some minor fallout, which was small enough not
> to warrant further rebasing and tweaking of an already much-rewritten series.
> 
> I will push in 72 hours.  Reviews welcome.
> 
> Regards,
>   Stefano
> 
> Stefano Lattarini (10):
>   [01] parallel-tests: simplify, using pattern rules with multiple targets
>   [02] cleanup: simplify %transforms for parallel-tests handling
>   [03] parallel-tests: simplify automake-time preprocessing
>   [04] coverage: parallel-tests and dynamic $(TESTS) content
>   [05] parallel-tests: support "TESTS = $(srcdir)/foo.test"
>   [06] build: define $(TESTS) through a wildcard
>   [07] tests: run long-running ones early (useful with parallel make)
>   [08] parallel-tests: remove obsolete comment
>   [09] parallel-tests: fix confusing function names in Makefile recipes
>   [10] tests: fix a spurious failure on Solaris 10
> 
>  .gitignore                                         |    2 +-
>  Makefile.am                                        |   93 +-
>  NG-NEWS                                            |   33 +
>  automake.in                                        |  133 +-
>  bootstrap                                          |    6 +-
>  doc/automake.texi                                  |    9 +-
>  gen-testsuite-part                                 |   78 +-
>  lib/Automake/Rule.pm                               |   39 +-
>  lib/am/check.am                                    |  181 ++-
>  lib/am/check2.am                                   |   13 +-
>  syntax-checks.mk                                   |    4 -
>  t/CheckListOfTests.am                              |   68 --
>  t/check7.sh                                        |    9 +-
>  t/exeext4.sh                                       |    1 +
>  t/list-of-tests.mk                                 | 1272 
> --------------------
>  t/parallel-tests-dynamic.sh                        |  145 +++
>  t/parallel-tests-exeext.sh                         |    3 -
>  t/parallel-tests-fd-redirect-exeext.sh             |    7 -
>  t/parallel-tests-fd-redirect.sh                    |    6 -
>  t/parallel-tests-fork-bomb.sh                      |    3 +-
>  t/parallel-tests-longest-stem.sh                   |   66 +
>  t/parallel-tests-no-repeat.sh                      |   44 +
>  t/parallel-tests-srcdir-in-test-name.sh            |   86 ++
>  t/parallel-tests-unreadable.sh                     |   65 +-
>  t/parallel-tests8.sh                               |   18 +-
>  t/self-check-dir.tap                               |    2 +-
>  t/serial-tests.sh                                  |    4 +-
>  t/spy-pattern-rules.sh                             |   54 +
>  t/tap-bad-prog.tap                                 |   13 +-
>  t/tap-basic.sh                                     |    2 +
>  t/tap-common-setup.sh                              |    8 +-
>  t/tap-diagnostic-custom.sh                         |    1 +
>  t/tap-more.sh                                      |    3 +-
>  t/tap-recheck.sh                                   |    3 +-
>  t/tap-signal.tap                                   |    2 +-
>  t/test-driver-custom-multitest-recheck.sh          |    2 +-
>  t/test-driver-custom-multitest-recheck2.sh         |    2 +-
>  t/test-driver-custom-multitest.sh                  |    2 +
>  t/test-driver-custom-no-extra-driver.sh            |    5 +-
>  t/test-driver-custom-xfail-tests.sh                |    2 +-
>  t/test-driver-fail.sh                              |    8 +-
>  t/test-driver-strip-vpath.sh                       |   11 +-
>  t/test-extensions.sh                               |    2 +-
>  t/test-metadata-global-log.sh                      |    1 +
>  t/test-metadata-global-result.sh                   |   45 +-
>  t/test-metadata-results.sh                         |   25 +-
>  t/test-missing2.sh                                 |   54 -
>  t/test-trs-basic.sh                                |    5 +-
>  ...-registered.sh => test-trs-recover-parallel.sh} |   62 +-
>  t/test-trs-recover2.sh                             |  133 --
>  50 files changed, 867 insertions(+), 1968 deletions(-)
>  delete mode 100644 t/CheckListOfTests.am
>  delete mode 100644 t/list-of-tests.mk
>  create mode 100755 t/parallel-tests-dynamic.sh
>  create mode 100755 t/parallel-tests-longest-stem.sh
>  create mode 100755 t/parallel-tests-no-repeat.sh
>  create mode 100755 t/parallel-tests-srcdir-in-test-name.sh
>  create mode 100755 t/spy-pattern-rules.sh
>  delete mode 100755 t/test-missing2.sh
>  rename t/{test-driver-trs-suffix-registered.sh => 
> test-trs-recover-parallel.sh} (52%)
>  delete mode 100755 t/test-trs-recover2.sh
> 
Merged to 'ng/master', and pushed.

Regards,
  Stefano