diff --git a/cli/app/list.go b/cli/app/list.go index d9cb5cd59..c6ebee4d9 100644 --- a/cli/app/list.go +++ b/cli/app/list.go @@ -176,26 +176,28 @@ can take some time. appStats.AutoUpdate = autoUpdate var newUpdates []string - parsedVersion, err := tagcmp.Parse(version) - if err != nil { - logrus.Warning("Can't parse ", app.Recipe, " version ", version, " ", err) - } - if ((version != "unknown") && (err == nil)) { - updates, err := recipe.GetRecipeCatalogueVersions(app.Recipe, catl) + if (version != "unknown") { + parsedVersion, err := tagcmp.Parse(version) + if (err != nil) { + logrus.Warning("Can't parse ", app.Recipe, " version ", version, " ", err) + } else { + updates, err := recipe.GetRecipeCatalogueVersions(app.Recipe, catl) - if err != nil { - logrus.Fatal(err) - } - - for _, update := range updates { - parsedUpdate, err := tagcmp.Parse(update) if err != nil { - logrus.Warning("can't parse ", app.Recipe," update version ", update, " ", err) + logrus.Fatal(err) } - if update != version && parsedUpdate.IsGreaterThan(parsedVersion) { - newUpdates = append(newUpdates, update) + for _, update := range updates { + parsedUpdate, err := tagcmp.Parse(update) + if err != nil { + logrus.Warning("can't parse ", app.Recipe," update version ", update, " ", err) + continue + } + + if update != version && parsedUpdate.IsGreaterThan(parsedVersion) { + newUpdates = append(newUpdates, update) + } } } }