Re: [PATCH v2] commit: add commit.signoff config option

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 25, 2015, 16:01 UTC
Message-ID: <xmqq4mlv22we.fsf@gitster.dls.corp.google.com>
In-Reply-To: <CADstYCamTxaLnZeYUma6t=KP3Fx2rQH3svqgCYP4WpAR9k_USg@mail.gmail.com>

Caio Marcelo de Oliveira Filho <cmarcelo@gmail.com> writes:

> A similar concern probably applies to format.signOff option. Would
> be sufficient to add a note about conscious act (like
> format.signOff has) to the config description?

I am generally in negative on automating this. This is not just you but makes everybody else's S-o-b less and less meaningful ("Their tool have ways to add that string randomly, and many of these ways the user ends up adding that string without even conciously thinking what they are doing. Does that string even mean anything anymore?").

One solution might be to make this not a straight-forward boolean, but an option that superficially takes a string and treats one specific value as "true" and everything else as "false", e.g.

[commit]
    signoff = "I certify that all my work is licenseable under DCO"

I dunno.