From: Andrew Gerrand <adg@golang.org>
Date: Thu, 19 Feb 2015 21:34:30 +0000 (+1100)
Subject: cmd/dist: show friendlier error message when building outside a Git repo
X-Git-Tag: go1.5beta1~1942
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=5f8423844463b3b77588e46ea57f44d8b69a1564;p=gostls13.git

cmd/dist: show friendlier error message when building outside a Git repo

Fixes #9932

Change-Id: I7943470a1784278a5c6e99c3b66c59d4953734ba
Reviewed-on: https://go-review.googlesource.com/5340
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
---

diff --git a/src/cmd/dist/build.go b/src/cmd/dist/build.go
index 3932c0bc42..141d3c9660 100644
--- a/src/cmd/dist/build.go
+++ b/src/cmd/dist/build.go
@@ -296,6 +296,11 @@ func findgoversion() string {
 		return chomp(readfile(path))
 	}
 
+	// Show a nicer error message if this isn't a Git repo.
+	if !isGitRepo() {
+		fatal("FAILED: not a Git repo; must put a VERSION file in $GOROOT")
+	}
+
 	// Otherwise, use Git.
 	// What is the current branch?
 	branch := chomp(run(goroot, CheckExit, "git", "rev-parse", "--abbrev-ref", "HEAD"))
@@ -321,6 +326,22 @@ func findgoversion() string {
 	return tag
 }
 
+// isGitRepo reports whether the working directory is inside a Git repository.
+func isGitRepo() bool {
+	p := ".git"
+	for {
+		fi, err := os.Stat(p)
+		if os.IsNotExist(err) {
+			p = filepath.Join("..", p)
+			continue
+		}
+		if err != nil || !fi.IsDir() {
+			return false
+		}
+		return true
+	}
+}
+
 /*
  * Initial tree setup.
  */