Re: [GSoC PATCH] submodule: warn on valueless active config

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 14, 2026, 19:14 UTC
Message-ID: <xmqqecg0ms5g.fsf@gitster.g>
In-Reply-To: <CABB4Jh1fUXKNn483FjD2S6U4cYVMEP6z+fjWMi8XRT+NQdNnYw@mail.gmail.com>

Tilak Raaz <raaztilak07@gmail.com> writes:

> The config parser previously threw a hard error if 'submodule.active'
> was provided without a value, causing commands to abort.

The standard helper to use is config_error_nonbool() when you need to report a section.variable defined this way

	[section]
		variable

without "= value", and section.variable cannot be a Boolean true.

The patch seems to be heavily whitespace damaged, and cannot be used, though.

Thanks.