From 089f0f9d9ed37d44ad2d8d0eb574e1496b113579 Mon Sep 17 00:00:00 2001 From: Sitaram Chamarty Date: Mon, 31 Dec 2012 06:23:00 +0530 Subject: [PATCH] on removing a repo... Not following through on instructions to remove a repo, per [1], is not sufficient. Even if you did just the first step, the repo should no longer be accessible. See [2] for discussion. As a bonus, we get rid of one pesky warning that always confused people. (In hindsight -- this confusion itself should have been a warning that something is wrong and needed fixing!) [1]: http://sitaramc.github.com/gitolite/repos.html [2]: http://groups.google.com/group/gitolite/browse_thread/thread/a3d4c3e917056abb --- src/lib/Gitolite/Conf/Load.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/Gitolite/Conf/Load.pm b/src/lib/Gitolite/Conf/Load.pm index 441c644..6c133e2 100644 --- a/src/lib/Gitolite/Conf/Load.pm +++ b/src/lib/Gitolite/Conf/Load.pm @@ -246,7 +246,7 @@ sub load_1 { } if ( -f "gl-conf" ) { - _warn "split conf not set, gl-conf present for '$repo'" if not $split_conf{$repo}; + return if not $split_conf{$repo}; my $cc = "./gl-conf"; _die "parse '$cc' failed: " . ( $! or $@ ) unless do $cc;