Re: [PATCH v5 00/17] cruft packs

From: Taylor Blau <me@ttaylorr.com>
Date: May 25, 2022, 21:09 UTC
Message-ID: <Yo6bDC8uivC3gM2o@nand.local>
In-Reply-To: <7f5a6a6a-c554-c659-72a8-404bc39e08c7@github.com>

On Wed, May 25, 2022 at 03:59:24PM -0400, Derrick Stolee wrote:

> I'd much rather have a consistent and proven way of specifying the
> hash value (using the oid_version() helper) than to try and make a
> new mechanism.

To be clear, I absolutely don't think any of us should have the attitude of repeating past bad decisions for the sake of consistency.

As best I can tell, our (Jonathan and I's) disagreement is on whether using "1" and "2" to identify which hash function is used by the .mtimes file is OK or not. I happen to think that it is acceptable, so the choice to continue to adopt this pattern was motivated by being consistent with a pattern that is good and works.

Thanks, Taylor

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.