Re: [RFC PATCH] repack: make repack -a equivalent to repack -A and drop previous -a behavior

From: Björn Steinbrink <b.steinbrink@gmx.de>
Date: Nov 14, 2008, 00:02 UTC
Message-ID: <20081114000219.GB5285@atjola.homenet>
In-Reply-To: <7xh1_tFsJkHTjg4Hjm-R4mGSRfYyGFmLI62OMmaNg32f86EbVIYvYrGiYpEGkvooY4pUM1e7CHk@cipher.nrlssc.navy.mil>

On 2008.11.13 17:22:36 -0600, Brandon Casey wrote:

Show 17 quoted lines

> Once upon a time, repack had only a single option which began with the first
> letter of the alphabet.  Then, a second was created which would repack
> unreachable objects into the newly created pack so that git-gc --auto could
> be invented.  But, the -a option was still necessary so that it could be
> called every now and then to discard the unreachable objects that were being
> repacked over and over and over into newly generated packs. Later, -A was
> changed so that instead of repacking the unreachable objects, it ejected
> them from the pack so that they resided in the object store in loose form,
> to be garbage collected by prune-packed according to normal expiry rules.
> 
> And so, -a lost its raison d'etre.
> 
> Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
> ---
> 
> 
> This is on top of bc/maint-keep-pack

I didn't check all the (proposed) commits for that branch, so just let me know if I'm missing anything, but doesn't this change mean that you just lose what "-ad" did?

We have:
	-a	Create a new pack, containing all reachable objects
	-A	Same as -a
	-ad	Same as -a, and drop all old packs and loose objects
	-Ad	Sama as -ad, but keep unreachable objects loose

-Ad is nice regarding it's safety-net value, but eg. after a large filter-branch run, when refs/original and the reflogs have been cleaned, you just want to get rid of all those old unreachable objects, immediately. For example after importing and massaging some large history from SVN, the -Ad behaviour is definitely _not_ what I want there. Writing a few thousand loose objects just to prune them is just a waste of time.

Björn