diff --git a/src/builder/builder.go b/src/builder/builder.go index 050b99c..78cab59 100644 --- a/src/builder/builder.go +++ b/src/builder/builder.go @@ -182,7 +182,7 @@ func (b *Builder) UpdateRepo(ver string, allbundles bool) { URL := "https://github.com/clearlinux/clr-bundles/archive/" + ver + ".tar.gz" err := helpers.Download(repo, URL) if err != nil { - fmt.Println("ERROR: Failed to download new clr-bundles, make sure the version is valid") + fmt.Fprintf(os.Stderr, "ERROR: Failed to download clr-bundles, make sure the version is valid: %s\n", err) os.Exit(1) } diff --git a/src/helpers/helpers.go b/src/helpers/helpers.go index a1c9a6e..e0184ef 100644 --- a/src/helpers/helpers.go +++ b/src/helpers/helpers.go @@ -171,14 +171,16 @@ func CopyFile(dest string, src string) error { func Download(filename string, url string) (err error) { infile, err := http.Get(url) if err != nil { - PrintError(err) return err } defer infile.Body.Close() + if infile.StatusCode != http.StatusOK { + return fmt.Errorf("Get %s replied: %d (%s)", url, infile.StatusCode, http.StatusText(infile.StatusCode)) + } + out, err := os.Create(filename) if err != nil { - PrintError(err) return err } defer out.Close() @@ -186,7 +188,6 @@ func Download(filename string, url string) (err error) { _, err = io.Copy(out, infile.Body) if err != nil { os.RemoveAll(filename) - PrintError(err) return err }