# git-gui PATCH Keep repo_config(gui.recentrepos) and .gitconfig in synch

2 messages from 2010-01-23 to 2010-01-23. Participants: Christopher Beelby, Shawn O. Pearce.
Thread: https://gitlist.dev/t/22359

## Christopher Beelby, 2010-01-23 19:21

Subject: git-gui PATCH Keep repo_config(gui.recentrepos) and .gitconfig in synch
Message-ID: <564f37c41001231121o10f88b6cwc35a126a0e79d3fc@mail.gmail.com>
URL: https://gitlist.dev/e/564f37c41001231121o10f88b6cwc35a126a0e79d3fc%40mail.gmail.com
In-Reply-To: <564f37c41001231118m3f253259g8876ac4fb2b927c2@mail.gmail.com>

```
This patch addresses two issues:

1. When the number of recent repo's gets to ten there can be a
situation where an item is removed from the .gitconfig file via
a call to git config --unset, but the internal representation of
that file (repo_config(gui.recentrepo)) is not updated. Then a
subsequent attempt to remove an item from the list fails because
git-gui attempts to call --unset on a value that has already been
removed. This leads to duplicates in the .gitconfig file, which
then also cause errors if the git-gui tries to --unset them (rather
than using --unset-all. --unset-all is not used because it is not
expected that duplicates should ever be allowed to exist.)

For complete step-by-step instructions on how to cause these situations
in git-gui see my comment on the msysgit project at
http://code.google.com/p/msysgit/issues/detail?id=362&colspec=ID%20Type%20Status%20Priority%20Component%20Owner%20Summary#c9

2. When loading the list of recent repositories (proc _get_recentrepos)
if a repo in the list is not considered a valid git reposoitory
then we should go ahead and remove it so it doesn't take up a slot
in the list (since we limit to 10 items). This will prevent a bunch of
invalid entries in the list (which are not shown) from making valid
entries dissapear off the list even when there are less than ten valid
entries.


diff --git a/git-gui/lib/choose_repository.tcl b/git-gui/lib/choose_repository.tcl
index 633cc57..3f8f303 100644
--- a/git-gui/lib/choose_repository.tcl
+++ b/git-gui/lib/choose_repository.tcl
@@ -235,6 +235,8 @@ proc _get_recentrepos {} {
 	foreach p [get_config gui.recentrepo] {
 		if {[_is_git [file join $p .git]]} {
 			lappend recent $p
+		} else {
+			_unset_recentrepo $p
 		}
 	}
 	return [lsort $recent]
@@ -243,6 +245,7 @@ proc _get_recentrepos {} {
 proc _unset_recentrepo {p} {
 	regsub -all -- {([()\[\]{}\.^$+*?\\])} $p {\\\1} p
 	git config --global --unset gui.recentrepo "^$p\$"
+	load_config 1
 }
 
 proc _append_recentrepos {path} {
@@ -261,6 +264,7 @@ proc _append_recentrepos {path} {
 
 	lappend recent $path
 	git config --global --add gui.recentrepo $path
+	load_config 1
 
 	while {[llength $recent] > 10} {
 		_unset_recentrepo [lindex $recent 0]

```

## Shawn O. Pearce, 2010-01-23 22:39

Subject: Re: git-gui PATCH Keep repo_config(gui.recentrepos) and .gitconfig in synch
Message-ID: <20100123223943.GC12679@spearce.org>
URL: https://gitlist.dev/e/20100123223943.GC12679%40spearce.org
In-Reply-To: <564f37c41001231121o10f88b6cwc35a126a0e79d3fc@mail.gmail.com>

```
Christopher Beelby <chris@celabs.com> wrote:
> This patch addresses two issues:

Thanks, applied.  In the future please follow the SubmittingPatches
guidelines, which includes sending the patch inline in your message
using the formatting created by `git format-patch`.
 
-- 
Shawn.

```