]> git.ipfire.org Git - thirdparty/git.git/commit - builtin/index-pack.c
t: invert `GIT_TEST_WRITE_REV_INDEX`
authorTaylor Blau <me@ttaylorr.com>
Wed, 12 Apr 2023 22:20:36 +0000 (18:20 -0400)
committerJunio C Hamano <gitster@pobox.com>
Thu, 13 Apr 2023 14:55:46 +0000 (07:55 -0700)
commit9f7f10a282d8adeb9da0990aa0eb2adf93a47ca7
tree844b9d5fc9a5bafd9b14e1e2758d57d52b9b082a
parenta8dd7e05b1c033917b900ea3d930e79eea3ff9a4
t: invert `GIT_TEST_WRITE_REV_INDEX`

Back in e8c58f894b (t: support GIT_TEST_WRITE_REV_INDEX, 2021-01-25), we
added a test knob to conditionally enable writing a ".rev" file when
indexing a pack. At the time, this was used to ensure that the test
suite worked even when ".rev" files were written, which served as a
stress-test for the on-disk reverse index implementation.

Now that reading from on-disk ".rev" files is enabled by default, the
test knob `GIT_TEST_WRITE_REV_INDEX` no longer has any meaning.

We could get rid of the option entirely, but there would be no
convenient way to test Git when ".rev" files *aren't* in place.

Instead of getting rid of the option, invert its meaning to instead
disable writing ".rev" files, thereby running the test suite in a mode
where the reverse index is generated from scratch.

This ensures that, when GIT_TEST_NO_WRITE_REV_INDEX is set to some
spelling of "true", we are still running and exercising Git's behavior
when forced to generate reverse indexes from scratch. Do so by setting
it in the linux-TEST-vars CI run to ensure that we are maintaining good
coverage of this now-legacy code.

Signed-off-by: Taylor Blau <me@ttaylorr.com>
Acked-by: Derrick Stolee <derrickstolee@github.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/index-pack.c
builtin/pack-objects.c
ci/run-build-and-tests.sh
pack-revindex.h
t/README
t/t5325-reverse-index.sh