]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: don't add detail to errPrintedOutput
authorRuss Cox <rsc@golang.org>
Fri, 16 Mar 2012 20:35:16 +0000 (16:35 -0400)
committerRuss Cox <rsc@golang.org>
Fri, 16 Mar 2012 20:35:16 +0000 (16:35 -0400)
This makes the last error-reporting CL a bit less
aggressive.  errPrintedOutput is a sentinel value
that should not be wrapped.

R=gri
CC=golang-dev
https://golang.org/cl/5845052

src/cmd/go/build.go

index 67b682ecb7798293bf69ff080d65ebffc773fb20..c1833e60752f84afa95f5cbf67d92983123e7c10 100644 (file)
@@ -607,7 +607,7 @@ func (b *builder) do(root *action) {
 // build is the action for building a single package or command.
 func (b *builder) build(a *action) (err error) {
        defer func() {
-               if err != nil {
+               if err != nil && err != errPrintedOutput {
                        err = fmt.Errorf("go build %s: %v", a.p.ImportPath, err)
                }
        }()
@@ -761,7 +761,7 @@ func (b *builder) build(a *action) (err error) {
 // install is the action for installing a single package or executable.
 func (b *builder) install(a *action) (err error) {
        defer func() {
-               if err != nil {
+               if err != nil && err != errPrintedOutput {
                        err = fmt.Errorf("go install %s: %v", a.p.ImportPath, err)
                }
        }()