[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Automake-ng] [PATCH 02/10] [ng] cleanup: simplify %transforms for paral
From: |
Stefano Lattarini |
Subject: |
[Automake-ng] [PATCH 02/10] [ng] cleanup: simplify %transforms for parallel-tests handling |
Date: |
Mon, 23 Apr 2012 11:57:20 +0200 |
* automake.in (handle_tests, handle_per_suffix_test): Drop some %transforms
previously required when processing 'lib/am/check.am': '%SOURCE%', '%OBJ%',
'%BASE%' and '?GENERIC?'.
Signed-off-by: Stefano Lattarini <address@hidden>
---
automake.in | 26 +++++++-------------------
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git a/automake.in b/automake.in
index 19c99c7..64ec53d 100644
--- a/automake.in
+++ b/automake.in
@@ -4706,23 +4706,21 @@ sub handle_tests_dejagnu
$output_rules .= file_contents ('dejagnu', new Automake::Location);
}
-sub handle_per_suffix_test
+sub handle_per_suffix_test($)
{
- my ($test_suffix, %transform) = @_;
- my ($pfx, $generic, $parallel_tests_option, $am_exeext);
+ my $test_suffix = shift;
+ my ($pfx, $parallel_tests_option, $am_exeext);
prog_error ("called with 'parallel-tests' option not set")
unless $parallel_tests_option = option 'parallel-tests';
if ($test_suffix eq '')
{
$pfx = '';
- $generic = 0;
}
else
{
prog_error ("test suffix '$test_suffix' lacks leading dot")
unless $test_suffix =~ m/^\.(.*)/;
$pfx = uc ($1) . '_';
- $generic = 1;
}
$am_exeext = exists $configure_vars{'EXEEXT'} ? 'am__EXEEXT'
: 'FALSE';
@@ -4739,11 +4737,9 @@ sub handle_per_suffix_test
INTERNAL);
}
$output_rules .= file_contents ('check2', new Automake::Location,
- GENERIC => $generic,
PFX => $pfx,
EXT => $test_suffix,
- am__EXEEXT => $am_exeext,
- %transform);
+ am__EXEEXT => $am_exeext);
}
# is_valid_test_extension ($EXT)
@@ -4851,13 +4847,8 @@ sub handle_tests
return substr ($obj, 0, length ($obj) - length
($test_suffix)) . '.log'
if substr ($obj, - length ($test_suffix)) eq $test_suffix;
}
- my $base = $obj;
- $obj .= '.log';
- handle_per_suffix_test ('',
- OBJ => $obj,
- BASE => $base,
- SOURCE => $val);
- return $obj;
+ handle_per_suffix_test ('');
+ return "$obj.log";
});
my $nhelper=1;
@@ -4882,10 +4873,7 @@ sub handle_tests
$nhelper++;
if ($test_suffix ne $at_exeext && $test_suffix ne '')
{
- handle_per_suffix_test ($test_suffix,
- OBJ => '',
- BASE => '$*',
- SOURCE => '$<');
+ handle_per_suffix_test ($test_suffix);
}
}
$clean_files{'$(TEST_LOGS)'} = MOSTLY_CLEAN;
--
1.7.9.5
- [Automake-ng] [PATCH 00/10] Rewrite parallel-tests to rely on GNU make capabilities, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 02/10] [ng] cleanup: simplify %transforms for parallel-tests handling,
Stefano Lattarini <=
- [Automake-ng] [PATCH 05/10] [ng] parallel-tests: support "TESTS = $(srcdir)/foo.test", Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 04/10] [ng] coverage: parallel-tests and dynamic $(TESTS) content, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 01/10] [ng] parallel-tests: simplify, using pattern rules with multiple targets, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 03/10] [ng] parallel-tests: simplify automake-time preprocessing, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 07/10] [ng] tests: run long-running ones early (useful with parallel make), Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 06/10] [ng] build: define $(TESTS) through a wildcard, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 08/10] [ng] parallel-tests: remove obsolete comment, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 09/10] [ng] parallel-tests: fix confusing function names in Makefile recipes, Stefano Lattarini, 2012/04/23
- [Automake-ng] [PATCH 10/10] [ng] tests: fix a spurious failure on Solaris 10, Stefano Lattarini, 2012/04/23
- Re: [Automake-ng] [PATCH 00/10] Rewrite parallel-tests to rely on GNU make capabilities, Stefano Lattarini, 2012/04/26