aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDerrick Stolee <derrickstolee@github.com>2024-09-20 00:00:23 +0000
committerJunio C Hamano <gitster@pobox.com>2024-09-20 14:44:32 -0700
commitb9183b0a02c59d86b6d9e4e0da996c8909ed6fa8 (patch)
tree5f48ffa7cf197c806d0441db2b7706bb6bbb8b4c
parent4f5551957ddd6c679fc8499d4e1e110d43a49071 (diff)
downloadgit-b9183b0a02c59d86b6d9e4e0da996c8909ed6fa8.tar.xz
scalar: configure maintenance during 'reconfigure'
The 'scalar reconfigure' command is intended to update registered repos with the latest settings available. However, up to now we were not reregistering the repos with background maintenance. In particular, this meant that the background maintenance schedule would not be updated if there are improvements between versions. Be sure to register repos for maintenance during the reconfigure step. Signed-off-by: Derrick Stolee <stolee@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--scalar.c3
-rwxr-xr-xt/t9210-scalar.sh7
2 files changed, 8 insertions, 2 deletions
diff --git a/scalar.c b/scalar.c
index 1fe8a93e65..62352e3d67 100644
--- a/scalar.c
+++ b/scalar.c
@@ -723,6 +723,9 @@ static int cmd_reconfigure(int argc, const char **argv)
the_repository = old_repo;
+ if (toggle_maintenance(1) >= 0)
+ succeeded = 1;
+
loop_end:
if (!succeeded) {
res = -1;
diff --git a/t/t9210-scalar.sh b/t/t9210-scalar.sh
index a41b4fcc08..67fc01e331 100755
--- a/t/t9210-scalar.sh
+++ b/t/t9210-scalar.sh
@@ -176,8 +176,11 @@ test_expect_success 'scalar reconfigure' '
scalar reconfigure one &&
test true = "$(git -C one/src config core.preloadIndex)" &&
git -C one/src config core.preloadIndex false &&
- scalar reconfigure -a &&
- test true = "$(git -C one/src config core.preloadIndex)"
+ rm one/src/cron.txt &&
+ GIT_TRACE2_EVENT="$(pwd)/reconfigure" scalar reconfigure -a &&
+ test_path_is_file one/src/cron.txt &&
+ test true = "$(git -C one/src config core.preloadIndex)" &&
+ test_subcommand git maintenance start <reconfigure
'
test_expect_success 'scalar reconfigure --all with includeIf.onbranch' '