Re: [PATCH] pack-objects: name pack files after trailer hash
- From
Jeff King <peff@peff.net>
- Date
- Dec 16, 2013, 19:35 UTC
- Message-ID
- <20131216193533.GA3488@sigill.intra.peff.net>
- In-Reply-To
- <xmqqzjo0oako.fsf@gitster.dls.corp.google.com>
On Mon, Dec 16, 2013 at 11:33:11AM -0800, Junio C Hamano wrote:
Show 12 quoted lines
> > <base-name> to determine the name of the created file.
> > When this option is used, the two files are written in
> > <base-name>-<SHA-1>.{pack,idx} files. <SHA-1> is a hash
> > + of the bytes of the packfile, and is written to the standard
>
> "hash of the bytes of the packfile" tempts one to do
>
> $ sha1sum .git/objects/pack/pack-*.pack
>
> but that is not what we expect. I wonder if there are better ways to
> phrase it (or alternatively perhaps we want to make that expectation
> hold by updating our code to hash)?Yeah, I wondered about that, but didn't think it was worth the verbosity to explain that the true derivation. I think Jonathan's suggestion takes care of it, though.
-Peff