On Wed Sep 18, 2024 at 2:32 PM AEST, Patrick Steinhardt wrote: > diff --git a/refs/reftable-backend.c b/refs/reftable-backend.c > index 1c4b19e737f..e90ddfb98dd 100644 > --- a/refs/reftable-backend.c > +++ b/refs/reftable-backend.c > @@ -256,6 +256,9 @@ static int reftable_be_config(const char *var, const char *value, > if (factor > UINT8_MAX) > die("reftable geometric factor cannot exceed %u", (unsigned)UINT8_MAX); > opts->auto_compaction_factor = factor; > + } else if (!strcmp(var, "reftable.locktimeout")) { > + unsigned long lock_timeout = git_config_ulong(var, value, ctx->kvi); > + opts->lock_timeout_ms = lock_timeout; > } Do we need to support the `0` and `-1` values that are possible for the "core.filesRefLockTimeout" and "core.packedRefsTimeout" timeouts here as well?