Commit f338d9f7 authored by Andrew Gerrand's avatar Andrew Gerrand

goinstall: undo 1ad616fb313d (always rebuild...)

CL 4627051 is a better way of doing the same thing.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/4654045
parent 524d02cb
......@@ -150,7 +150,7 @@ func logPackage(pkg string) {
}
// install installs the package named by path, which is needed by parent.
func install(pkg, parent string) (built bool) {
func install(pkg, parent string) {
// Make sure we're not already trying to install pkg.
switch visit[pkg] {
case done:
......@@ -201,12 +201,9 @@ func install(pkg, parent string) (built bool) {
errorf("%s: package has no files\n", pkg)
return
}
var depBuilt bool
for _, p := range dirInfo.Imports {
if p != "C" {
if install(p, pkg) {
depBuilt = true
}
install(p, pkg)
}
}
if errors {
......@@ -227,13 +224,12 @@ func install(pkg, parent string) (built bool) {
script.Clean()
}
if *doInstall {
if depBuilt || script.Stale() {
if script.Stale() {
printf("%s: install\n", pkg)
if err := script.Run(); err != nil {
errorf("%s: install: %v\n", pkg, err)
return
}
built = true
} else {
printf("%s: up-to-date\n", pkg)
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment