summaryrefslogtreecommitdiff
path: root/builtin
diff options
context:
space:
mode:
authorDavid Turner <dturner@twosigma.com>2016-12-28 17:45:42 -0500
committerJunio C Hamano <gitster@pobox.com>2016-12-29 13:45:37 -0800
commit1c409a705cb30ae3db6cdd48757c4a85f51456d4 (patch)
tree31094619358b4e3634660cc2c7a7aacc786d27d0 /builtin
parentbdf56de896b48d6fe3ad90e92138b5fd6579748d (diff)
downloadgit-1c409a705cb30ae3db6cdd48757c4a85f51456d4.tar.gz
repack: die on incremental + write-bitmap-indexdt/disable-bitmap-in-auto-gc
The bitmap index only works for single packs, so requesting an incremental repack with bitmap indexes makes no sense. Signed-off-by: David Turner <dturner@twosigma.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
-rw-r--r--builtin/repack.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/builtin/repack.c b/builtin/repack.c
index 80dd06b4a2..677bc7c81a 100644
--- a/builtin/repack.c
+++ b/builtin/repack.c
@@ -18,6 +18,12 @@ static const char *const git_repack_usage[] = {
NULL
};
+static const char incremental_bitmap_conflict_error[] = N_(
+"Incremental repacks are incompatible with bitmap indexes. Use\n"
+"--no-write-bitmap-index or disable the pack.writebitmaps configuration."
+);
+
+
static int repack_config(const char *var, const char *value, void *cb)
{
if (!strcmp(var, "repack.usedeltabaseoffset")) {
@@ -206,6 +212,9 @@ int cmd_repack(int argc, const char **argv, const char *prefix)
if (pack_kept_objects < 0)
pack_kept_objects = write_bitmaps;
+ if (write_bitmaps && !(pack_everything & ALL_INTO_ONE))
+ die(_(incremental_bitmap_conflict_error));
+
packdir = mkpathdup("%s/pack", get_object_directory());
packtmp = mkpathdup("%s/.tmp-%d-pack", packdir, (int)getpid());