From d947dd966b1b951c8f3b7d588932fa59eeddce9c Mon Sep 17 00:00:00 2001 From: Cassie Recher Date: Thu, 8 Aug 2019 16:06:53 -0700 Subject: [PATCH] Remove strings.Builder usage from fetch.go. This broke compatibility with Go 1.9. --- claat/fetch/fetch.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/claat/fetch/fetch.go b/claat/fetch/fetch.go index 6ec5a4d1a..7ab04bf3b 100644 --- a/claat/fetch/fetch.go +++ b/claat/fetch/fetch.go @@ -172,16 +172,16 @@ func (f *Fetcher) SlurpImages(src, dir string, steps []*types.Step) (map[string] } imap := make(map[string]string, count) - errBuilder := strings.Builder{} + var errStr string for i := 0; i < count; i++ { r := <-ch imap[r.file] = r.url if r.err != nil { - errBuilder.WriteString(fmt.Sprintf("%s => %s: %v\n", r.url, r.file, r.err)) + errStr += fmt.Sprintf("%s => %s: %v\n", r.url, r.file, r.err) } } - if errBuilder.Len() > 0 { - return nil, errors.New(errBuilder.String()) + if len(errStr) > 0 { + return nil, errors.New(errStr) } return imap, nil