You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by cc...@apache.org on 2016/12/07 00:14:37 UTC
incubator-mynewt-newt git commit: MYNEWT-504 newt - Ignore error when
repo not inst.
Repository: incubator-mynewt-newt
Updated Branches:
refs/heads/develop 8e6024c5a -> 2828be955
MYNEWT-504 newt - Ignore error when repo not inst.
The compatibility check was failing when a repo hadn't been installed
yet, causing newt to terminate. This check is not critical, and should
not cause newt to abort.
Project: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/commit/2828be95
Tree: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/tree/2828be95
Diff: http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/diff/2828be95
Branch: refs/heads/develop
Commit: 2828be9557d4050b9e1c22a0435362c11f81fb8e
Parents: 8e6024c
Author: Christopher Collins <cc...@apache.org>
Authored: Tue Dec 6 16:13:19 2016 -0800
Committer: Christopher Collins <cc...@apache.org>
Committed: Tue Dec 6 16:13:19 2016 -0800
----------------------------------------------------------------------
newt/repo/repo.go | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-mynewt-newt/blob/2828be95/newt/repo/repo.go
----------------------------------------------------------------------
diff --git a/newt/repo/repo.go b/newt/repo/repo.go
index 83ff137..fe4a5bc 100644
--- a/newt/repo/repo.go
+++ b/newt/repo/repo.go
@@ -615,7 +615,13 @@ func (r *Repo) Init(repoName string, rversreq string, d downloader.Downloader) e
upToDate, err := r.HasMinCommit()
if err != nil {
- return err
+ // If there is an error checking the repo's commit log, just abort
+ // the check. An error could have many possible causes: repo not
+ // installed, network issue, etc. In none of these cases does it
+ // makes sense to warn about an out of date repo. If the issue is
+ // a real one, it will be handled when it prevents forward
+ // progress.
+ return nil
}
if !upToDate {