From 55c9d50706aa55130a66b071e260c9252d3dcf11 Mon Sep 17 00:00:00 2001
From: Graham Christensen <graham@grahamc.com>
Date: Mon, 4 Dec 2017 19:13:34 -0500
Subject: [PATCH] Only gist on failures

---
 ofborg/src/tasks/massrebuilder.rs | 42 +++++++++++++++----------------
 1 file changed, 21 insertions(+), 21 deletions(-)

diff --git a/ofborg/src/tasks/massrebuilder.rs b/ofborg/src/tasks/massrebuilder.rs
index d61a248..5c3d660 100644
--- a/ofborg/src/tasks/massrebuilder.rs
+++ b/ofborg/src/tasks/massrebuilder.rs
@@ -270,35 +270,35 @@ impl worker::SimpleWorker for MassRebuildWorker {
                      status.set(hubcaps::statuses::State::Pending);
 
                      let state: hubcaps::statuses::State;
-                     let mut out: File;
+                     let gist_url: Option<String>;
                      match check.execute(Path::new(&refpath)) {
-                         Ok(o) => {
-                             out = o;
+                         Ok(_) => {
                              state = hubcaps::statuses::State::Success;
+                             gist_url = None;
                          }
-                         Err(o) => {
-                             out = o;
+                         Err(mut out) => {
                              state = hubcaps::statuses::State::Failure;
+
+                             let mut files = HashMap::new();
+                             files.insert(check.name(),
+                                          hubcaps::gists::Content {
+                                              filename: Some(check.name()),
+                                              content: file_to_str(&mut out),
+                                          }
+                             );
+
+                             gist_url = Some(gists.create(
+                                 &hubcaps::gists::GistOptions {
+                                     description: Some(format!("{:?}", state)),
+                                     public: Some(true),
+                                     files: files,
+                                 }
+                             ).expect("Failed to create gist!").html_url);
                          }
                      }
 
-                     let mut files = HashMap::new();
-                     files.insert(check.name(),
-                                  hubcaps::gists::Content {
-                                      filename: Some(check.name()),
-                                      content: file_to_str(&mut out),
-                                  }
-                     );
 
-                     let gist_url = gists.create(
-                         &hubcaps::gists::GistOptions {
-                             description: Some(format!("{:?}", state)),
-                             public: Some(true),
-                             files: files,
-                         }
-                     ).expect("Failed to create gist!").html_url;
-
-                     status.set_url(Some(gist_url));
+                     status.set_url(gist_url);
                      status.set(state.clone());
 
                      if state == hubcaps::statuses::State::Success {