Skip to content

Commit

Permalink
on removing a repo...
Browse files Browse the repository at this point in the history
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 089f0f9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/lib/Gitolite/Conf/Load.pm
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 089f0f9

Please sign in to comment.