Re: [PATCH 0/3] batch blob diff generation
- From
Jeff King <peff@peff.net>
- Date
- Dec 20, 2024, 09:10 UTC
- Message-ID
- <20241220091048.GE133148@coredump.intra.peff.net>
- In-Reply-To
- <20241220090908.GD133148@coredump.intra.peff.net>
On Fri, Dec 20, 2024 at 04:09:08AM -0500, Jeff King wrote:
Show 21 quoted lines
> > Ah, OK. So your "caution" was about the underlying mechanism to > > allow attributes corrected from the specified tree, and not > > specifically about using "attr.tree" to specify that tree? That was > > what got me confused. > > > > If that is the case, I do not think the documentation patch that > > started this exchange that adds attr.tree to where GIT_ATTR_SOURCE > > and --attr-source are already mentioned makes anything worse. > > Yeah, I agree it's somewhat orthogonal. Your patch made me think about > it because it is advertising the config variant more widely. Somebody > doing: > > git --attr-source=foo diff ... > > is probably OK, but: > > git --attr-source=foo pack-objects ... > > is less so. Using attr.tree instead means you're going to do the latter > whether you intended to or not.
Re-reading my message, I guess I didn't really give any conclusion. ;)
I should add: I'm OK with leaving the performance implications undocumented for now. Hopefully in the long run somebody is interested in addressing them.
-Peff