From 6f1d00097be403998e5490845835ecb23d2c741c Mon Sep 17 00:00:00 2001
From: Daniel Wagner-Hall <daniel@matrix.org>
Date: Thu, 19 Nov 2015 16:15:13 -0500
Subject: [PATCH] Only bother trying to fetch if we need to

---
 scripts/speculator/main.go | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/scripts/speculator/main.go b/scripts/speculator/main.go
index 380cd2bc..4972210e 100644
--- a/scripts/speculator/main.go
+++ b/scripts/speculator/main.go
@@ -147,12 +147,20 @@ func (s *server) updateBase() error {
 	return runGitCommand(s.matrixDocCloneURL, []string{"fetch"})
 }
 
+func (s *server) knowsAbout(sha string) bool {
+	s.mu.Lock()
+	defer s.mu.Unlock()
+	return runGitCommand(s.matrixDocCloneURL, []string{"cat-file", "-e", sha + "^{commit}"}) == nil
+}
+
 // generateAt generates spec from repo at sha.
 // Returns the path where the generation was done.
 func (s *server) generateAt(sha string) (dst string, err error) {
-	err = s.updateBase()
-	if err != nil {
-		return
+	if !s.knowsAbout(sha) {
+		err = s.updateBase()
+		if err != nil {
+			return
+		}
 	}
 	s.mu.Lock()
 	dst, err = gitClone(s.matrixDocCloneURL, true)