Permalink
Browse files

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
  • Loading branch information...
sitaramc committed Dec 31, 2012
1 parent 5aef1ad commit 089f0f9d9ed37d44ad2d8d0eb574e1496b113579
Showing with 1 addition and 1 deletion.
  1. +1 −1 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;

0 comments on commit 089f0f9

Please sign in to comment.