@@ -954,8 +954,8 @@ END
</ul>
<h2>Steps</h2>
<table rules=all>
-<tr><th>stepno</th><th>testid</th><th>script</th><th>status</th>
-<th>started</th><th>duration</th></tr>
+<tr><th>stepno</th><th>testid</th><th>script</th><th>started</th>
+<th>status</th><th>duration</th></tr>
END
my %issteplog;
@@ -977,6 +977,8 @@ END
$stepcol->('testid', '');
$stepcol->('step', '');
+ printf H "<td>%s</td>", show_abs_time $step->{started};
+
my $st= $step->{status};
my $logfilename= "$step->{stepno}.$step->{step}.log";
printf H "<td bgcolor='%s'><a href='%s'>%s</a></td>",
@@ -985,8 +987,6 @@ END
encode_entities($st);
$issteplog{$logfilename}= 1;
- printf H "<td>%s</td>", show_abs_time $step->{started};
-
if (defined $step->{finished} && defined $step->{started}) {
printf H "<td align=right>%d</td>\n",
$step->{finished} - $step->{started};
With the current ordering of status => started I frequently (more often than not) read the failing step as "(stepno,testid,script) failed at <time>" (where <time> is actually the start time, not the fail time). Move the "started" column to the left of the "status" column. On the basis that "(stepno,testid,script) started at $time and failed" reads more (chrono)logically. Signed-off-by: Ian Campbell <ian.campbell@citrix.com> --- sg-report-flight | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)