From be60237c1c6fccc50d30b78ed0406534871ae485 Mon Sep 17 00:00:00 2001
From: Jeff Carr <jcarr@wit.com>
Date: Fri, 2 Feb 2024 17:56:59 -0600
Subject: [PATCH] further automation

Signed-off-by: Jeff Carr <jcarr@wit.com>
---
 globalDisplayOptions.go |  1 +
 releaseWindow.go        | 24 +++++++++++++++++-------
 2 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/globalDisplayOptions.go b/globalDisplayOptions.go
index 1f6b149..88d1217 100644
--- a/globalDisplayOptions.go
+++ b/globalDisplayOptions.go
@@ -55,6 +55,7 @@ func globalDisplayOptions(box *gui.Node) {
 		scanForReady()
 		scanForReady()
 		reposwin.Toggle()
+		findNextDirty("")
 	})
 
 	me.autoHideReadOnly = group1.NewCheckbox("Hide read-only repos").SetChecked(true)
diff --git a/releaseWindow.go b/releaseWindow.go
index d7f388e..2372fc4 100644
--- a/releaseWindow.go
+++ b/releaseWindow.go
@@ -307,6 +307,7 @@ func createReleaseBox(box *gui.Node) {
 		buttonEnable()
 	})
 
+	/*
 	release.grid.NewButton("CheckPrimativeGoMod()", func() {
 		tmp := release.current.String()
 		log.Info("Run CheckGoSum on repo:", tmp)
@@ -317,6 +318,13 @@ func createReleaseBox(box *gui.Node) {
 		}
 
 	})
+	*/
+
+	release.makeRedomodB = release.grid.NewButton("make redomod", func() {
+		buttonDisable()
+		release.current.status.MakeRedomod()
+		buttonEnable()
+	})
 
 	release.grid.NewButton("goodCheckGoSum()", func() {
 		buttonDisable()
@@ -327,11 +335,6 @@ func createReleaseBox(box *gui.Node) {
 		}
 		buttonEnable()
 	})
-	release.makeRedomodB = release.grid.NewButton("make redomod", func() {
-		buttonDisable()
-		release.current.status.MakeRedomod()
-		buttonEnable()
-	})
 }
 
 func goodCheckGoSum() bool {
@@ -347,17 +350,24 @@ func goodCheckGoSum() bool {
 		repo, ok := me.allrepos[depname]
 		if ok {
 			goSumS := repo.getGoSumStatus()
+			// ignore dependencies on whitelisted repos
+			// TODO: warn the user about the whitelist
 			if goSumS == "WHITELIST" {
 				continue
 			}
-			if release.current.status.ReadOnly() {
+			// check if the dependent repo is ReadOnly
+			// if so, there isn't anything we can do about
+			// version mis-matches
+			if repo.status.ReadOnly() {
 				continue
 			}
 			lastS := repo.status.GetLastTagVersion()
 			targetS := repo.targetVersion.String()
 			log.Info("   repo deps:", depname, version, "vs", goSumS, lastS, targetS)
 			if lastS != targetS {
-				log.Info("   repo deps: FALSE lastS != targetS", lastS, targetS)
+				log.Info("repo deps: FALSE lastS != targetS", lastS, targetS)
+				log.Info("repo deps: FALSE status.ReadOnly()", release.current.status.ReadOnly())
+				log.Info("repo deps: FALSE path", release.current.String())
 				maybe = false
 			}
 		} else {