Merge pull request ethereum#3317 from fjl/build-unstable-simplify

karalabe · web-flow · commit e05d35e6e04f · 2016-11-23T12:24:28.000+02:00
build, internal/build: simplify unstable build checks
diff --git a/appveyor.yml b/appveyor.yml
@@ -22,19 +22,18 @@ environment:
 
 install:
   - rmdir C:\go /s /q
-  - appveyor DownloadFile https://storage.googleapis.com/golang/go1.7.3.windows-amd64.zip
-  - 7z x go1.7.3.windows-amd64.zip -y -oC:\ > NUL
+  - appveyor DownloadFile https://storage.googleapis.com/golang/go1.7.3.windows-%GETH_ARCH%.zip
+  - 7z x go1.7.3.windows-%GETH_ARCH%.zip -y -oC:\ > NUL
   - go version
   - gcc --version
 
 build_script:
-  - go run build\ci.go install -arch %GETH_ARCH%
+  - go run build\ci.go install
 
 after_build:
-  - go run build\ci.go archive -arch %GETH_ARCH% -type zip -signer WINDOWS_SIGNING_KEY -upload gethstore/builds
-  - go run build\ci.go nsis -arch %GETH_ARCH% -signer WINDOWS_SIGNING_KEY -upload gethstore/builds
+  - go run build\ci.go archive -type zip -signer WINDOWS_SIGNING_KEY -upload gethstore/builds
+  - go run build\ci.go nsis -signer WINDOWS_SIGNING_KEY -upload gethstore/builds
 
 test_script:
-  - set GOARCH=%GETH_ARCH%
   - set CGO_ENABLED=1
   - go run build\ci.go test -vet -coverage
diff --git a/build/ci.go b/build/ci.go
@@ -459,7 +459,7 @@ func makeWorkdir(wdflag string) string {
 }
 
 func isUnstableBuild(env build.Environment) bool {
-	if env.Branch != "master" && env.Tag != "" {
+	if env.Tag != "" {
 		return false
 	}
 	return true
diff --git a/internal/build/env.go b/internal/build/env.go
@@ -88,8 +88,9 @@ func LocalEnv() Environment {
 			env.Branch = b
 		}
 	}
-	// Note that we don't get the current git tag. It would slow down
-	// builds and isn't used by anything.
+	if env.Tag == "" {
+		env.Tag = RunGit("for-each-ref", "--points-at=HEAD", "--count=1", "--format=%(refname:short)", "refs/tags")
+	}
 	return env
 }
 
diff --git a/internal/build/util.go b/internal/build/util.go
@@ -76,14 +76,19 @@ func VERSION() string {
 	return string(bytes.TrimSpace(version))
 }
 
+var warnedAboutGit bool
+
 // RunGit runs a git subcommand and returns its output.
 // The command must complete successfully.
 func RunGit(args ...string) string {
 	cmd := exec.Command("git", args...)
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout, cmd.Stderr = &stdout, &stderr
 	if err := cmd.Run(); err == exec.ErrNotFound {
-		log.Println("no git in PATH")
+		if !warnedAboutGit {
+			log.Println("Warning: can't find 'git' in PATH")
+			warnedAboutGit = true
+		}
 		return ""
 	} else if err != nil {
 		log.Fatal(strings.Join(cmd.Args, " "), ": ", err, "\n", stderr.String())

Original file line number	Diff line number	Diff line change
`@@ -459,7 +459,7 @@ func makeWorkdir(wdflag string) string {`
`459`	`459`	`}`
`460`	`460`
`461`	`461`	`func isUnstableBuild(env build.Environment) bool {`
`462`		`- if env.Branch != "master" && env.Tag != "" {`
	`462`	`+ if env.Tag != "" {`
`463`	`463`	`return false`
`464`	`464`	`}`
`465`	`465`	`return true`
Original file line number	Diff line number	Diff line change
`@@ -88,8 +88,9 @@ func LocalEnv() Environment {`
`88`	`88`	`env.Branch = b`
`89`	`89`	`}`
`90`	`90`	`}`
`91`		`- // Note that we don't get the current git tag. It would slow down`
`92`		`- // builds and isn't used by anything.`
	`91`	`+ if env.Tag == "" {`
	`92`	`+ env.Tag = RunGit("for-each-ref", "--points-at=HEAD", "--count=1", "--format=%(refname:short)", "refs/tags")`
	`93`	`+ }`
`93`	`94`	`return env`
`94`	`95`	`}`
`95`	`96`