[Automake-NG] [FYI] [ng] tests: enhance spy tests on patter rules

* t/spy-pattern-rules.sh : Fix heading comments to be more faithful about what the test really checks. Add more tests checking that, in case of overlapping pattern rules, the ones with the shortest stem wins. Signed-off-by: Stefano Lattarini --- t/spy-pattern-rules.sh | 43 +++

Re: [Automake-NG] [PATCH 00/33] Several renamings of private make variables

On 07/21/2012 06:10 PM, Akim Demaille wrote: > > Le 21 juil. 2012 à 14:55, Stefano Lattarini a écrit : > >> The first bach of "am__foo -> am.foo" renamings. See also this recent >> patch that announced the upcoming renamings: >>

Re: [Automake-NG] [PATCH 0/2] Two simple refactorings

On 07/21/2012 06:31 PM, Akim Demaille wrote: > > Le 21 juil. 2012 à 16:14, Stefano Lattarini a écrit : > >> Will push these shortly. >> >> Stefano Lattarini (2): >> automake: new function &almost_verbatim (small refactoring) >> automake: inline '&handle_install' > > Sure. > Thanks, pushed. R

Re: [Automake-NG] [PATCH 0/2] Two simple refactorings

Le 21 juil. 2012 à 16:14, Stefano Lattarini a écrit : > Will push these shortly. > > Stefano Lattarini (2): > automake: new function &almost_verbatim (small refactoring) > automake: inline '&handle_install' Sure.

Re: [Automake-NG] [PATCH 00/33] Several renamings of private make variables

Le 21 juil. 2012 à 14:55, Stefano Lattarini a écrit : > The first bach of "am__foo -> am.foo" renamings. See also this recent > patch that announced the upcoming renamings: > > > Stefano Lattarini (33): > [ng] rename: am__di

automake-ng@gnu.org

* automake.in (handle_install): Delete, inlined ... (generate_makefile): ... in here, as it had become a trivial one-liner. Signed-off-by: Stefano Lattarini --- automake.in |9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/automake.in b/automake.in index b1875fd..e52

[Automake-NG] [PATCH 1/2] automake: new function &almost_verbatim (small refactoring)

* automake.in (almost_verbatim): New function, thin wrapper around '&preprocess_file'. It requires just the name of the fragment to preprocess (assuming it is one provided by automake, and thus taking care to prepend the "$libdir/am/" path and append the ".am" extension automatically), and then au

[Automake-NG] [PATCH 0/2] Two simple refactorings

Will push these shortly. Stefano Lattarini (2): automake: new function &almost_verbatim (small refactoring) automake: inline '&handle_install' automake.in | 36 1 file changed, 20 insertions(+), 16 deletions(-) -- 1.7.10.4

[Automake-NG] [PATCH 33/33] [ng] rename: am__relativize -> am.dist.relativize-path

Signed-off-by: Stefano Lattarini --- lib/am/distdir.am |6 +++--- t/relativize.tap |8 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/am/distdir.am b/lib/am/distdir.am index 39c819e..d16cc1d 100644 --- a/lib/am/distdir.am +++ b/lib/am/distdir.am @@ -101,7 +10

[Automake-NG] [PATCH 32/33] [ng] rename: am__canon -> am.util.canon

Signed-off-by: Stefano Lattarini --- lib/am/check-typos.am |6 +++--- lib/am/header-vars.am |2 +- t/internals.tap | 28 ++-- 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/lib/am/check-typos.am b/lib/am/check-typos.am index 5b19111..e52e5

[Automake-NG] [PATCH 31/33] [ng] rename: am__toupper -> am.util.toupper

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am|4 ++-- lib/am/parallel-tests.am |2 +- t/internals.tap | 30 +++--- 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 1035e

[Automake-NG] [PATCH 29/33] [ng] rename: am__using_parallel_tests -> am.conf.using-parallel-tests

Signed-off-by: Stefano Lattarini --- lib/am/check-typos.am|2 +- lib/am/parallel-tests.am |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/check-typos.am b/lib/am/check-typos.am index 224d71d..5b19111 100644 --- a/lib/am/check-typos.am +++ b/lib/am/check-ty

[Automake-NG] [PATCH 30/33] [ng] rename: am__tolower -> am.util.tolower

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am|4 ++-- t/internals.tap | 30 +++--- t/test-extensions-dynamic.sh |2 +- 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am

[Automake-NG] [PATCH 28/33] [ng] rename: am__vpath_rewrite -> am.vpath.rewrite

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |2 +- t/vpath-rewrite.sh|2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 691c505..4eca206 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am

[Automake-NG] [PATCH 27/33] [ng] rename: am__handle_exeext -> am.conf.handle-exeext

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am|4 ++-- lib/am/parallel-tests.am |4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 4d5531b..691c505 100644 --- a/lib/am/header-vars.am +++ b/lib/am/head

[Automake-NG] [PATCH 26/33] [ng] rename: am__private_suffix -> am.hack.private-suffix

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 57db26d..4d5531b 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -249,14 +249,14 @@ am__mem

[Automake-NG] [PATCH 25/33] [ng] rename: am__strip_suffixes -> am.util.strip-suffixes

Signed-off-by: Stefano Lattarini --- lib/am/check-typos.am|2 +- lib/am/header-vars.am|6 +++--- lib/am/parallel-tests.am |2 +- t/internals.tap | 48 +++--- 4 files changed, 29 insertions(+), 29 deletions(-) diff --git a/lib

[Automake-NG] [PATCH 23/33] [ng] rename: am__uniq -> am.util.uniq

Signed-off-by: Stefano Lattarini --- lib/am/compile.am |2 +- lib/am/distdir.am |2 +- lib/am/header-vars.am |2 +- t/internals.tap | 32 4 files changed, 19 insertions(+), 19 deletions(-) diff --git a/lib/am/compile.am b/lib/am/compil

[Automake-NG] [PATCH 24/33] [ng] am__strip_suffixes: use $0 for recursive invocations

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index d369de1..4304560 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -254,7 +254,7 @@ am__strip_suffi

[Automake-NG] [PATCH 22/33] [ng] am__uniq: use $0 for recursive invocations

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index af2c6ef..450bec4 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -158,7 +158,7 @@ am__uniq = $(st

[Automake-NG] [PATCH 21/33] [ng] rename: am__strip_lastword -> am.util.strip-last-word

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |6 +++--- t/internals.tap | 18 +- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index ef25345..af2c6ef 100644 --- a/lib/am/header-vars.am +++ b

[Automake-NG] [PATCH 20/33] [ng] rename: am__strip_firstword -> am.util.strip-first-word

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- t/internals.tap | 18 +- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index c0e15ba..ef25345 100644 --- a/lib/am/header-vars.am +++ b/l

[Automake-NG] [PATCH 18/33] [ng] rename: am__empty -> am.chars.empty

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |6 +++--- t/clean-many.sh |6 +++--- t/clean-many2.sh |2 +- t/dist-many.sh|5 ++--- t/dist-many2.sh |5 ++--- t/memoize.tap |4 ++-- 6 files changed, 13 insertions(+), 15 deletion

[Automake-NG] [PATCH 19/33] [ng] rename: am__make_dryrun -> am.make.dry-run

Signed-off-by: Stefano Lattarini --- lib/am/all-target.am |2 +- lib/am/check-typos.am |2 +- lib/am/distdir.am |2 +- lib/am/header-vars.am |2 +- lib/am/lisp.am|2 +- t/make-dryrun.tap |6 +++--- 6 files changed, 8 insertions(+), 8 deletions(-) diff --g

[Automake-NG] [PATCH 17/33] [ng] cosmetics: fix an imprecise comment

* lib/am/header-vars.am (am__empty): About this. Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index ad96691..c6ad4d3 100644 --- a/lib/am/header-vars.am +++ b/li

[Automake-NG] [PATCH 16/33] [ng] rename: am__squote -> am.chars.squote

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 71c0d5f..ad96691 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -47,7 +47,7 @@ am.chars.lpar

[Automake-NG] [PATCH 15/33] [ng] rename: am__dquote -> am.chars.dquote

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index ed5f0d2..71c0d5f 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -46,7 +46,7 @@ am.chars.hash

[Automake-NG] [PATCH 14/33] [ng] rename: am__bquote -> am.chars.bquote

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 9065e73..ed5f0d2 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -45,7 +45,7 @@ am.chars.doll

[Automake-NG] [PATCH 13/33] [ng] rename: am__rparen -> am.chars.rparen

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 5562472..9065e73 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -44,7 +44,7 @@ am.chars.comm

[Automake-NG] [PATCH 12/33] [ng] rename: am__lparen -> am.chars.lparen

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 43340c0..5562472 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -43,7 +43,7 @@ am.chars.bsla

[Automake-NG] [PATCH 11/33] [ng] rename: am__pound -> am.chars.hash

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 3aea065..43340c0 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -42,7 +42,7 @@ endef am.cha

[Automake-NG] [PATCH 00/33] Several renamings of private make variables

The first bach of "am__foo -> am.foo" renamings. See also this recent patch that announced the upcoming renamings: Stefano Lattarini (33): [ng] rename: am__dist_* -> am.dist.* [ng] rename: am__all_progs -> am.all-progs [n

[Automake-NG] [PATCH 10/33] [ng] rename: am__dollar -> am.chars.dollar

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 5dd223a..3aea065 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -41,7 +41,7 @@ endef # to m

[Automake-NG] [PATCH 09/33] [ng] rename: am__comma -> am.chars.comma

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 65db607..5dd223a 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -40,7 +40,7 @@ endef # Some

[Automake-NG] [PATCH 08/33] [ng] rename: am__bslash -> am.chars.bslash

Signed-off-by: Stefano Lattarini --- lib/am/header-vars.am |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am index 7236305..65db607 100644 --- a/lib/am/header-vars.am +++ b/lib/am/header-vars.am @@ -39,7 +39,7 @@ endef # So

[Automake-NG] [PATCH 01/33] [ng] rename: am__dist_* -> am.dist.*

Here is the complete list of renames: am__dist_common-> am.dist.common-files am__dist_sources -> am.dist.sources am__dist_files -> am.dist.all-files am__dist_files_1 -> am.dist.files-tmp1 am__dist_files_2 -> am.dist.files-tmp2 am_

[Automake-NG] [PATCH 07/33] [ng] rename: am__seen_error -> am.error.seen

Signed-off-by: Stefano Lattarini --- automake.in |2 +- lib/am/header-vars.am |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/automake.in b/automake.in index 08637c0..7dcee9c 100644 --- a/automake.in +++ b/automake.in @@ -6986,7 +6986,7 @@ sub generate_ma

[Automake-NG] [PATCH 06/33] [ng] cosmetics: improve a comment

* automake.in (generate_makefile): In here, about the generated Makefile baling out if '$(am__seen_error)' has been set by some previous function call. Signed-off-by: Stefano Lattarini --- automake.in |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/automake.in b/automa

[Automake-NG] [PATCH 05/33] [ng] rename: am__error -> am.error

Signed-off-by: Stefano Lattarini --- lib/am/check-typos.am|4 ++-- lib/am/header-vars.am|2 +- lib/am/parallel-tests.am |2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/am/check-typos.am b/lib/am/check-typos.am index ebc0402..b752b59 100644 --- a/lib/a

[Automake-NG] [PATCH 04/33] [ng] rename: am__all_ltlibs -> am.all-ltlibs

Signed-off-by: Stefano Lattarini --- automake.in |2 +- lib/am/check-typos.am |2 +- t/all-prog-libs.sh|4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/automake.in b/automake.in index c890497..197945a 100644 --- a/automake.in +++ b/automake.in @@ -

[Automake-NG] [PATCH 03/33] [ng] rename: am__all_libs -> am.all-libs

Signed-off-by: Stefano Lattarini --- automake.in |2 +- lib/am/check-typos.am |2 +- t/all-prog-libs.sh|6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/automake.in b/automake.in index 5b9db39..c890497 100644 --- a/automake.in +++ b/automake.in @@

[Automake-NG] [PATCH 02/33] [ng] rename: am__all_progs -> am.all-progs

Signed-off-by: Stefano Lattarini --- automake.in|2 +- lib/am/check-typos.am |2 +- lib/am/serial-tests.am |2 +- t/all-prog-libs.sh |6 +++--- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/automake.in b/automake.in index 4b2faf9..5b9db39 100644 -

[Automake-NG] [FYI] [ng] tests: fix unportable $((...)) usage

* t/longline.sh: Use "$i", not bare "i", to reference the value of the shell variable 'i' in a $((...)) expression. Issue revealed by a failure with dash 0.5.2. Signed-off-by: Stefano Lattarini --- t/longline.sh |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/longline.s

[Automake-NG] [FYI] [ng] fixup: imprecise comment in a test

* t/dist-many2.sh: We don't fake the command line length for the shell at "max 2000 chars", but at at "max 4000 chars". Adjust heading comment accordingly. Signed-off-by: Stefano Lattarini --- t/dist-many2.sh |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/dist-many2.sh

[Automake-NG] [PATCH 6/6] [ng] recheck: don't exceed command line limits, even with many failed tests

Related to automake bug#7868. * lib/am/parallel-tests.sh (recheck): Arrange recursive make invocation to pass the list of tests to be rechecked in the make standard input rather than on the make command line. (am__test_bases): Only define if not already set, to make the new idiom referenced above

[Automake-NG] [PATCH 3/6] [ng] check: use awk rather than grep+xargs to count test results

* lib/am/parallel-tests.am ($(TEST_SUITE_LOG)): Here, with the help of ... (am__count_test_results): ... this new internal variable. Signed-off-by: Stefano Lattarini --- lib/am/parallel-tests.am | 76 +++--- 1 file changed, 58 insertions(+), 18 deletions

[Automake-NG] [PATCH 4/6] [ng] check: refactor for less duplication and better performances

* lib/am/parallel-tests.am (am__count_test_results): Adjust this awk program to emit a shell snippet to be executed by the calling recipe ... ($(TEST_SUITE_LOG)): ... here. This avoid the need to call the program in $(am__count_test_results) once for each valid test result. Signed-off-by: Stefano

[Automake-NG] [PATCH 5/6] [ng] coverage: recheck with many failed tests

* t/parallel-tests-many.sh: Extend to check that the 'recheck' target works even when a huge number of tests (~ 30 thousands) have failed in the previous testsuite run. Currently this doesn't work, and cause the test to fails, so that we list it ... * Makefile.am (XFAIL_TESTS): ... in here. Signe

[Automake-NG] [PATCH 2/6] [ng] parallel-tests: do not exceed command line length limits

Fixes automake bug#7868. Two things worth noting: + a "make recheck" issued after a huge number of tests have failed can still hit command-line length issues; + the check-recipes now contain (first among the Automake-generated recipes) a use of the 'xargs' utility. These issues will lik

[Automake-NG] [PATCH 1/6] [ng] coverage: testing with lots of test scripts

See long-standing automake bug#7868. * t/parallel-tests-many.sh: Simplify and enhance. Among the other things, this test now tries running ~ 30 thousands tests. Currently fails on several systems (e.g., Linux 2.6.30 on i686, Solaris 10 on i86pc). Signed-off-by: Stefano Lattarini --- t/paralle

[Automake-NG] [PATCH 0/6] Run many tests without hitting command line length limits

Stefano Lattarini (6): [ng] coverage: testing with lots of test scripts [ng] parallel-tests: do not exceed command line length limits [ng] check: use awk rather than grep+xargs to count test results [ng] check: refactor for less duplication and better performances [ng] coverage: recheck w