From f13ba7892045b73661c82cb621d32f97b29694c9 Mon Sep 17 00:00:00 2001 From: Alan Donovan Date: Thu, 14 Aug 2014 12:10:34 -0400 Subject: [PATCH] go.tools/go/loader: make error message for massive failure (e.g. no "fmt") more concise. LGTM=gri R=gri CC=golang-codereviews https://golang.org/cl/126080043 --- go/loader/loader.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/go/loader/loader.go b/go/loader/loader.go index 3800389f..d495105a 100644 --- a/go/loader/loader.go +++ b/go/loader/loader.go @@ -604,8 +604,13 @@ func (conf *Config) Load() (*Program, error) { } } if errpkgs != nil { - return nil, fmt.Errorf("couldn't load packages due to errors: %s", - strings.Join(errpkgs, ", ")) + var more string + if len(errpkgs) > 3 { + more = fmt.Sprintf(" and %d more", len(errpkgs)-3) + errpkgs = errpkgs[:3] + } + return nil, fmt.Errorf("couldn't load packages due to errors: %s%s", + strings.Join(errpkgs, ", "), more) } }