2009/7/20 Eric Wong <normalperson@xxxxxxxx>: > Robert Zeh <robert.a.zeh@xxxxxxxxx> wrote: >> Add a git svn gc command that gzips all unhandled.log files, and removes >> all index files under .git/svn. >> >> Signed-off-by: Robert Allan Zeh <robert.a.zeh@xxxxxxxxx> >> --- >> Documentation/git-svn.txt | 4 ++++ >> git-svn.perl | 37 +++++++++++++++++++++++++++++++++++++ >> t/t9140-git-svn-gc.sh | 45 ++++++++++++++++++++++++++++++++++++ >> +++++++++ >> 3 files changed, 86 insertions(+), 0 deletions(-) >> create mode 100755 t/t9140-git-svn-gc.sh > > Hi Robert, > > Your mailer is mangling whitespace badly so the patch isn't applying > at all. Make sure indents are real tabs like the rest of the code. > > Some more comments below, but I think this will be a good addition > to git svn. > >> --- a/git-svn.perl >> +++ b/git-svn.perl >> @@ -35,11 +35,14 @@ push @Git::SVN::Ra::ISA, 'SVN::Ra'; >> push @SVN::Git::Editor::ISA, 'SVN::Delta::Editor'; >> push @SVN::Git::Fetcher::ISA, 'SVN::Delta::Editor'; >> use Carp qw/croak/; >> +use Compress::Zlib; > > I'd "require" Compress::Zlib lazilly so it's not loaded at startup. > It's not a stock component of Perl and not needed for the majority of > commands. Actually, it has been a core component since 5.9.3 cheers, Yves -- perl -Mre=debug -e "/just|another|perl|hacker/" -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html