[OSSTEST PATCH 07/17] step logfiles: Break out $logfilename in two places

[Ian Jackson <ian.jackson@eu.citrix.com>]

No functional change.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 determine-failure-reasons | 4 ++--
 sg-report-flight          | 3 ++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/determine-failure-reasons b/determine-failure-reasons
index ecff332..a3b2100 100755
--- a/determine-failure-reasons
+++ b/determine-failure-reasons
@@ -52,8 +52,8 @@ sub gethosts ($) {
 }
 
 while (my $step= $stepsq->fetchrow_hashref) {
-    $step->{Log}= "$c{Logs}/$step->{flight}/$step->{job}/".
-        "$step->{stepno}.$step->{step}.log";
+    my $logfile = "$step->{stepno}.$step->{step}.log";
+    $step->{Log}= "$c{Logs}/$step->{flight}/$step->{job}/".$logfile;
 
     if (!stat $step->{Log}) {
         die "$step->{Log} $!" unless $!==&ENOENT;
diff --git a/sg-report-flight b/sg-report-flight
index fd8a68b..ec30ed0 100755
--- a/sg-report-flight
+++ b/sg-report-flight
@@ -1167,8 +1167,9 @@ END
 		: ('#ffff00',250);
 	}
 	my $ch = "bgcolor=\"$bgc\"";
+	my $logfilename= "$s->{stepno}.$s->{step}.log";
 	my $h = "<a href=\"".encode_entities($s->{job}).
-	    "/$s->{stepno}.".encode_entities($s->{step}).".log\">";
+	    "/".encode_entities($logfilename)."\">";
 	#print H "[$s->{stepno}] ";
 	$h .= encode_entities($sum_core);
 	$h .= "</a>";
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel