try to figure out wtf is going on

1 files changed, 19 insertions, 56 deletions

diff --git a/findNext.go b/findNext.go
index 261e47c..5434b7e 100644
--- a/findNext.go
+++ b/findNext.go
@@ -16,43 +16,6 @@ var findCounter int
 var findFix bool = false
 var findOk bool = true
 
-func checkDeps(repo *gitpb.Repo) error {
-	if repo.GoDeps == nil {
-		return fmt.Errorf("%s has GoDeps == nil", repo.GetNamespace())
-	}
-	for dep := range repo.GoDeps.IterAll() {
-		// log.Info(repo.GetNamespace(), dep.GoPath, dep.Version)
-
-		// check if the package in question is waiting for another package to publish
-		found := me.forge.FindByNamespace(dep.GoPath)
-		if found == nil {
-			return fmt.Errorf("%s has dep == nil", repo.GetNamespace(), dep.GoPath)
-		}
-
-		// loop through all the repos that need to be published with new go versions
-		all := me.found.SortByFullPath()
-		for all.Scan() {
-			check := all.Next()
-
-			if found.GetNamespace() == check.GetNamespace() {
-				// this package is waiting on other packages to publish
-				return fmt.Errorf("%s is waiting on %s", repo.GetNamespace(), found.GetNamespace())
-			}
-		}
-		// found package isn't being published. is the version correct?
-		// never check this? we are done?
-		/*
-			if found.GetLastTag() == dep.Version {
-				// everything is normal
-			} else {
-				return fmt.Errorf("%s version mismatch on %s (%s vs %s)", repo.GetNamespace(), found.GetNamespace(), found.GetLastTag(), dep.Version)
-			}
-		*/
-	}
-	// everything might be cool?
-	return nil
-}
-
 // trys to figure out if there is still something to update
 // todo: redo this logic as it is terrible
 // rename this findNext()
@@ -62,28 +25,28 @@ func findNext() bool {
 	for all.Scan() {
 		check := all.Next()
 		if check.GetMasterBranchName() != check.GetCurrentBranchName() {
-			log.Info("YOU MUST BE ON THE MASTER BRANCH", check.GetNamespace())
+			log.Info("FIND NEXT: YOU MUST BE ON THE MASTER BRANCH", check.GetNamespace())
 			continue
 		}
 		if check.IsDirty() {
-			log.Info("CAN NOT RELEASE DIRTY REPO", check.GetNamespace())
+			log.Info("FIND NEXT: CAN NOT RELEASE DIRTY REPO", check.GetNamespace())
 			continue
 		}
 		if alreadyDone(check) {
-			log.Info("findNext() alreadyDone. WHY IS THIS STILL CHECKING?", check.GetNamespace())
+			log.Info("FIND NEXT: findNext() alreadyDone. WHY IS THIS STILL CHECKING?", check.GetNamespace())
 			continue
 		}
-		log.Info("CHECKING START:", check.GetNamespace())
+		log.Info("FIND NEXT: CHECKING START:", check.GetNamespace())
 
 		if me.forge.Config.IsPrivate(check.GetNamespace()) {
-			log.Info("GOOD TO GO ON PRIVATE REPO", check.GetNamespace())
+			log.Info("FIND NEXT: GOOD TO GO ON PRIVATE REPO", check.GetNamespace())
 			setCurrentRepo(check, "should be good to release", "pretty sure")
 			return true
 		}
 
 		godepsNew, err := check.GoSumFromRepo()
 		if err != nil {
-			log.Info("CHECKING go deps from repo failed", err)
+			log.Info("FIND NEXT: CHECKING go deps from repo failed", err)
 			continue
 
 		}
@@ -93,41 +56,41 @@ func findNext() bool {
 			cmd := []string{"go", "mod", "tidy"}
 			err := check.RunVerbose(cmd)
 			if err != nil {
-				log.Info("go mod tidy failed. this go package needs to be examined by hand as it doesn't appear to be primitive")
+				log.Info("FIND NEXT: go mod tidy failed. this go package needs to be examined by hand as it doesn't appear to be primitive")
 				os.Exit(-1)
 			}
 			// if godepsNew == nil, then this go package is a primitive and there is no go.sum file
 		} else {
 			if err := testGoDepsCheckOk(godepsNew, argv.Verbose); err != nil {
-				log.Info("CHECKING current repo deps failed", err)
+				log.Info("FIND NEXT: CHECKING current repo deps failed", err)
 				continue
 			}
 		}
 
 		if err := me.forge.FinalGoDepsCheckOk(check, argv.Verbose); err != nil {
 			// if err := me.forge.FinalGoDepsCheckOk(check, false); err != nil {
-			log.Info("FinalGoDepsCheckOk() repo=", check.GetNamespace(), "err:", err)
-			log.Info("CHECKING END:", check.GetNamespace())
-			log.Info("")
+			log.Info("FIND NEXT: FinalGoDepsCheckOk() repo=", check.GetNamespace(), "err:", err)
+			log.Info("FIND NEXT: CHECKING END:", check.GetNamespace())
+			log.Info("FIND NEXT: ")
 			continue
 		}
-		log.Info("GOOD TO GO ON", check.GetNamespace())
+		log.Info("FIND NEXT: GOOD TO GO ON", check.GetNamespace())
 		setCurrentRepo(check, "should be good to release", "pretty sure")
 		return true
 	}
 
 	if findCounter == 0 {
-		log.Info("NOTHING TO UPDATE. findCounter =", findCounter, "found len =", me.found.Len())
+		log.Info("FIND NEXT: NOTHING TO UPDATE. findCounter =", findCounter, "found len =", me.found.Len())
 		if me.found.Len() == 0 {
 			printDone()
 			okExit("")
 		}
 	} else {
-		log.Info("me.current is nil findCounter =", findCounter, "so set findFix =", findFix)
+		log.Info("FIND NEXT: me.current is nil findCounter =", findCounter, "so set findFix =", findFix)
 	}
-	log.Info("tried to findNext() but not sure what to do next counter =", findCounter, "findFix =", findFix)
-	setCurrentRepo(nil, "findNext found nothing", "crap")
-	me.release.status.SetText("ALL DONE?")
+	log.Info("FIND NEXT: tried to findNext() but not sure what to do next counter =", findCounter, "findFix =", findFix)
+	setCurrentRepo(nil, "FIND NEXT: findNext found nothing", "crap")
+	me.release.status.SetText("FIND NEXT: ALL DONE?")
 	return false
 }
 
@@ -136,8 +99,8 @@ func setCurrentRepo(check *gitpb.Repo, s string, note string) bool {
 	if check == nil {
 		me.release.repo.SetText("")
 		me.release.version.SetText("")
-		me.release.releaseVersionB.SetText("nope")
-		me.release.version.SetText("badver")
+		me.release.releaseVersionB.SetText("repo == nil")
+		me.release.version.SetText("repo == nil")
 	} else {
 		me.release.repo.SetText(check.GetNamespace())
 		me.release.version.SetText(check.GetTargetVersion())