summaryrefslogtreecommitdiff
path: root/builtin/pack-objects.c
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2014-06-10 16:19:13 -0400
committerJunio C Hamano <gitster@pobox.com>2014-06-10 14:01:53 -0700
commit15a906c5e99fc34534fd40d61590da142dee71d2 (patch)
treec525d9831f29155faf63decab4c5fff09ee2d6e8 /builtin/pack-objects.c
parentd078d85bc3cd74bbe1f230feb26cbe28f29d6d25 (diff)
downloadgit-15a906c5e99fc34534fd40d61590da142dee71d2.tar.gz
pack-objects: stop respecting pack.writebitmaps
The handling of the pack.writebitmaps config option originally happened in pack-objects, which is quite low-level. It would make more sense for drivers of pack-objects to read the config, and then manipulate pack-objects with command-line options. Recently, repack learned to do so, making the low-level read of pack.writebitmaps redundant here. Other callers, like upload-pack, would not generally want to write bitmaps anyway. This could be considered a regression for somebody who is driving pack-objects themselves outside of repack and expects the config option to be used. However, such users seem rather unlikely given how new the bitmap code is (and the fact that they would basically be reimplementing repack in the first place). Note that we do not do anything with pack.writeBitmapHashCache here. That option is not about "do we write bimaps", but rather "when we are writing bitmaps, how do we do it?". You would want that to kick in anytime you decide to write them, similar to how pack.indexVersion is used. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/pack-objects.c')
-rw-r--r--builtin/pack-objects.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c
index fd741970e6..2175031699 100644
--- a/builtin/pack-objects.c
+++ b/builtin/pack-objects.c
@@ -2199,10 +2199,6 @@ static int git_pack_config(const char *k, const char *v, void *cb)
cache_max_small_delta_size = git_config_int(k, v);
return 0;
}
- if (!strcmp(k, "pack.writebitmaps")) {
- write_bitmap_index = git_config_bool(k, v);
- return 0;
- }
if (!strcmp(k, "pack.writebitmaphashcache")) {
if (git_config_bool(k, v))
write_bitmap_options |= BITMAP_OPT_HASH_CACHE;