cs-bisection-step has had, for a long time, code which is supposed to
handle the situation where the template flight contains build job
references to other flights.
However:
- The regexp to spot these other-flight job reference runvars would
never match because it said \s where \S was probably intended (and
. would be better);
- If it were to match, the flight and job arguments to the recursive
preparejob invocation were teh wrong way round. preparejob takes
the job name first.
Fix these two bugs. Now it does seem to work properly.
Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com>
---
cs-bisection-step | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/cs-bisection-step b/cs-bisection-step
index 37efbea..b676044 100755
--- a/cs-bisection-step
+++ b/cs-bisection-step
@@ -1190,8 +1190,8 @@ END
foreach my $subjob (@$subjobs) {
my $target;
my $jobspec= $subjob->{val};
- if ($jobspec =~ m/^(\d+)\.(\s+)$/) {
- $target= preparejob($1, $2, 1);
+ if ($jobspec =~ m/^(\d+)\.(.+)$/) {
+ $target= preparejob($2, $1, 1);
} else {
$target= preparejob($jobspec, $copyflight, 1);
}
--
1.7.10.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
