Re: [PATCH] Add git svn gc command

Robert Zeh [off-list ref] wrote:

Hi Eric,

Thanks for all of the suggestions.  I've attached a patch that takes all 
of the list suggestions into account.  I hope I've implemented the  
can_compress changes properly.

Hi Robert, that looks fine to me.

I believe I've fixed the white space issues; I tried a round trip and my 
tabs were preserved.  The getcwd stuff was indeed leftover code from 
testing; I found another left over debug statement.

The tabs seem alright, but spaces are still mangled.  Perhaps
Documentation/SubmittingPatches has tips for your mailer? (or check out
git send-email)

I have one remaining question ---  does git svn gc need to avoid  
removing index files if --no-metadata was set?  If it does, I'd  
appreciate a tip on how to see if it was set.

Nope, not at all, index files are very safely removable unless there's a
clone/fetch actively running.  The rev_map files on the other hand are
not removable iff --no-metadata or --use-svm-props is used.

-- 
Eric Wong