Reduce redundant tarball downloads

Author: supercaracal

bin/build

@@ -6,23 +6,29 @@ require 'digest/sha1'
 require 'fileutils'
 
 class Builder
+  TARBALL_CACHE_EXPIRATION = 60 * 10
+
   def initialize(redis_branch, tmp_dir)
     @redis_branch = redis_branch
     @tmp_dir = tmp_dir
     @build_dir = File.join(@tmp_dir, "cache", "redis-#{redis_branch}")
   end
 
   def run
-    download_tarball
+    download_tarball_if_needed
     if old_checkum != checksum
       build
       update_checksum
     end
     0
   end
 
-  def download_tarball
-    command!('wget', tarball_url, '-O', tarball_path)
+  private
+
+  def download_tarball_if_needed
+    return if File.exist?(tarball_path) && File.mtime(tarball_path) > Time.now - TARBALL_CACHE_EXPIRATION
+
+    command!('wget', '-q', tarball_url, '-O', tarball_path)
   end
 
   def tarball_path

makefile

@@ -63,7 +63,7 @@ stop_slave:
 	@$(call kill-redis,${SLAVE_PID_PATH})
 
 start_slave: ${BINARY}
-	${BINARY}\
+	@${BINARY}\
 		--daemonize  yes\
 		--pidfile    ${SLAVE_PID_PATH}\
 		--port       ${SLAVE_PORT}\