On Tue, 2016-08-30 at 09:54 -0700, Junio C Hamano wrote:
> Support for more generic footers was supposed to come when the
> "interpret-trailers" topic started, but the author of the topic
> seems to have lost interest before the mechanism has become ready to
> be integrated in the workflow commands like "am", "commit", "rebase"
> etc., which is unfortunate.
I think adding at least an option to git send-email
allowing auto-cc's for all
"<foo>-by: [name] <address>"
lines in the commit log would be useful.
Today, only "Signed-off-by" and "CC" lines are
added to cc's.
I've always called these lines "<foo>-by:" lines
"signatures", but perhaps there's a better name.
Any preference?
from git send-email --help
--suppress-cc=<category>
Specify an additional category of recipients to suppress the
auto-cc of:
· author will avoid including the patch author
· self will avoid including the sender
· cc will avoid including anyone mentioned in Cc lines in the
patch header except for self (use self for that).
· bodycc will avoid including anyone mentioned in Cc lines in the
patch body (commit message) except for self (use self for
that).
· sob will avoid including anyone mentioned in Signed-off-by
lines except for self (use self for that).
· cccmd will avoid running the --cc-cmd.
· body is equivalent to sob + bodycc
· all will suppress all auto cc values.
> >
> > sequencer.c:append_signoff() has a flag for APPEND_SIGNOFF_DEDUP
> Yes, I think this is one of the warts we talked about getting rid of
> but haven't got around to it. It is there because "format-patch -s"
> was incorrectly written to dedup Signed-off-by: from anywhere in its
> early implementation and to keep the same behaviour. We should drop
> that flag from append_signoff() function.