glock sync: only "go get" if the package doesn't yet exist.

2x speedup for sync on GOPATH that's all OK

sync.go

@@ -77,13 +77,16 @@ func runSync(cmd *Command, args []string) {
 		var importDir = filepath.Join(gopath(), "src", importPath)
 
 		// Try to find the repo.
-		// go get it beforehand just in case it doesn't exist. (no-op if it does exist)
-		// (ignore failures due to "no buildable files" or build errors in the package.)
-		var getOutput, _ = run("go", "get", "-v", "-d", importPath)
+		var getOutput []byte
 		var repo, err = glockRepoRootForImportPath(importPath)
 		if err != nil {
-			fmt.Println(string(getOutput)) // in case the get failed due to connection error
-			perror(err)
+			// go get it in case it doesn't exist. (no-op if it does exist)
+			// (ignore failures due to "no buildable files" or build errors in the package.)
+			getOutput, _ = run("go", "get", "-v", "-d", importPath)
+			repo, err = glockRepoRootForImportPath(importPath)
+			if err != nil {
+				perror(err)
+			}
 		}
 
 		var maybeGot = ""

util.go

@@ -11,17 +11,24 @@ import (
 
 // glockRepoRootForImportPath wraps the vcs.go version.  If the stock one
 // doesn't work, it looks for .git, .hg directories up to the tree.
+// This is done to support repos with non-go-get friendly names.
+// Also, returns an error if it doesn't exist (e.g. it needs to be go gotten).
 func glockRepoRootForImportPath(importPath string) (*repoRoot, error) {
-	var rr, err = repoRootForImportPath(importPath)
-	if err == nil {
-		return rr, nil
-	}
-
-	pkg, err := build.Import(importPath, "", build.FindOnly)
+	var pkg, err = build.Import(importPath, "", build.FindOnly)
 	if err != nil {
 		return nil, err
 	}
 
+	rr, err := repoRootForImportPath(importPath)
+	if err == nil {
+		// it may not exist, even with err == nil
+		_, err = os.Stat(pkg.Dir)
+		if err != nil {
+			return nil, err
+		}
+		return rr, nil
+	}
+
 	var dir = pkg.Dir
 	var rootImportPath = pkg.ImportPath
 	for len(dir) > 1 {