Merge branch 'master' of github.com:twitchtv/retool

retool_test.go

@@ -60,6 +60,21 @@ func TestRetool(t *testing.T) {
 			}
 		})
 
+		t.Run("sync", func(t *testing.T) {
+			dir, cleanup := setupTempDir(t)
+			defer cleanup()
+
+			runRetoolCmd(t, dir, retool, "add", "github.com/twitchtv/retool", "origin/master")
+
+			// Delete existing tools directory to try and trigger out of date
+			_ = os.RemoveAll(filepath.Join(dir, "_tools"))
+
+			// Should be able to sync
+			runRetoolCmd(t, dir, retool, "sync")
+
+			assertBinInstalled(t, dir, "retool")
+		})
+
 		t.Run("build", func(t *testing.T) {
 			t.Parallel()
 			dir, cleanup := setupTempDir(t)

sync.go

@@ -1,15 +1,12 @@
 package main
 
-import "os/exec"
+import "os"
 
 func (s spec) sync() {
 	m := getManifest()
 	if m.outOfDate(s.Tools) {
-		log("syncing")
-
 		// Delete existing tools directory
-		cmd := exec.Command("rm", "-r", "-f", toolDirPath)
-		_, err := cmd.Output()
+		err := os.RemoveAll(toolDirPath)
 		if err != nil {
 			fatalExec("failed to remove _tools ", err)
 		}