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 {