Subject: Re: [PATCH v5 3/3] fast-import: add mode to sign commits with invalid signatures

From: Junio C Hamano <[email-masked]>
Date: Thu, 12 Mar 2026 13:20:23 -0700

Justin Tobler <[email-masked]> writes:

> +	case SIGN_SIGN_IF_INVALID:
> +		if (subject_len > 100)
> +			warning(_("signing commit with invalid signature for '%.100s...'\n"
> +				  "  allegedly by %s"), subject, signer);
> +		else if (subject_len > 0)
> +			warning(_("signing commit with invalid signature for '%.*s'\n"
> +				  "  allegedly by %s"), subject_len, subject, signer);
> +		else
> +			warning(_("signing commit with invalid signature\n"
> +				  "  allegedly by %s"), signer);

A very minor point, but my reading hiccuped around these messages,
sounding as if we are adding an invalid signature to the commit.

Perhaps "replacing an invalid signature for commit" or "re-signing
commit that has an invalid signature" or along that lines would
reduce the chance of confusion?