Re: [PATCH v5 00/12] Hash-independent tests (part 3)
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Sep 13, 2018, 17:00 UTC
- Message-ID
- <abd21e68-ac72-3111-5813-7d47af3d6aa6@gmail.com>
- In-Reply-To
- <20180913051742.252276-1-sandals@crustytoothpaste.net>
On 9/13/2018 1:17 AM, brian m. carlson wrote:
Show 10 quoted lines
> This is the next in the series of improvements to make tests > hash-independent. > > A range-diff is below. > > Changes from v4: > * Add local statements to the &&-chain. > * Fix a typo in the local statement. > * Add a helpful comment about why test_detect_hash is hard-coded to > SHA-1.
The range-diff and these changes since v4 are reasonable. I'm happy with the current version.
I'm looking forward to this change getting merged down, as I also want to use 'test_oid rawsz' for the 'git multi-pack-index verify' tests that are currently under review (v2 imminent).
Thanks,
-Stolee