From c2c2633e501869139c5d8401814f0f9d736d2e1b Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <edolstra@gmail.com>
Date: Tue, 11 Apr 2017 14:25:48 +0200
Subject: [PATCH] Fix link to cached build log

Fixes #462.
---
 src/lib/Hydra/Controller/Build.pm | 5 ++++-
 src/root/build.tt                 | 9 +++++----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/lib/Hydra/Controller/Build.pm b/src/lib/Hydra/Controller/Build.pm
index d688ea0d..047d0d8e 100644
--- a/src/lib/Hydra/Controller/Build.pm
+++ b/src/lib/Hydra/Controller/Build.pm
@@ -73,7 +73,10 @@ sub build_GET {
     if ($build->finished && $build->iscachedbuild) {
         my $path = ($build->buildoutputs)[0]->path or die;
         my $cachedBuildStep = findBuildStepByOutPath($self, $c, $path);
-        $c->stash->{cachedBuild} = $cachedBuildStep->build if defined $cachedBuildStep;
+        if (defined $cachedBuildStep) {
+            $c->stash->{cachedBuild} = $cachedBuildStep->build;
+            $c->stash->{cachedBuildStep} = $cachedBuildStep;
+        }
     }
 
     if ($build->finished) {
diff --git a/src/root/build.tt b/src/root/build.tt
index 5284907c..d79283c1 100644
--- a/src/root/build.tt
+++ b/src/root/build.tt
@@ -137,7 +137,7 @@ END;
   [% IF isAggregate %]<li><a href="#tabs-constituents" data-toggle="tab">Constituents</a></li>[% END %]
   <li><a href="#tabs-details" data-toggle="tab">Details</a></li>
   <li><a href="#tabs-buildinputs" data-toggle="tab">Inputs</a></li>
-  [% IF steps %]<li><a href="#tabs-buildsteps" data-toggle="tab">Build steps</a></li>[% END %]
+  [% IF steps.size() > 0 %]<li><a href="#tabs-buildsteps" data-toggle="tab">Build steps</a></li>[% END %]
   [% IF build.dependents %]<li><a href="#tabs-usedby" data-toggle="tab">Used by</a></li>[% END%]
   [% IF drvAvailable %]<li><a href="#tabs-build-deps" data-toggle="tab">Build dependencies</a></li>[% END %]
   [% IF available %]<li><a href="#tabs-runtime-deps" data-toggle="tab">Runtime dependencies</a></li>[% END %]
@@ -230,9 +230,10 @@ END;
               <tr>
                 <th>Logfile:</th>
                 <td>
-                  <a class="btn btn-mini" href="[% c.uri_for('/build' actualBuild.id 'log') %]">pretty</a>
-                  <a class="btn btn-mini" href="[% c.uri_for('/build' actualBuild.id 'log' 'raw') %]">raw</a>
-                  <a class="btn btn-mini" href="[% c.uri_for('/build' actualBuild.id 'log' 'tail') %]">tail</a>
+                  [% actualLog = cachedBuildStep ? c.uri_for('/build' cachedBuild.id 'nixlog' cachedBuildStep.stepnr) : c.uri_for('/build' build.id 'log') %]
+                  <a class="btn btn-mini" href="[%actualLog%]">pretty</a>
+                  <a class="btn btn-mini" href="[%actualLog%]/raw">raw</a>
+                  <a class="btn btn-mini" href="[%actualLog%]/tail">tail</a>
                 </td>
               </tr>
             [% END %]