threads / patch / 62968

v2doc(send-email): use correct name of the --signed-off-by-cc in git send-email doc

Subject: [PATCH v2] doc(send-email): use correct name of the --signed-off-by-cc in git send-email doc

## tl;dr

4 messages between Feb 18, 2025 and Feb 5, 2026. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Matěj Cepl· Feb 18, 2025, 08:58 UTC · lore

I don’t think there is an option --signed-off-cc (without -by) for git send-email.

Signed-off-by: Matěj Cepl <mcepl@cepl.eu>
---
Rebase on the top of the current master (Whoops!) and improve the commit message.
 Documentation/git-send-email.adoc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to Documentation/git-send-email.adoc +1 −1
diff --git a/Documentation/git-send-email.adoc b/Documentation/git-send-email.adoc
index 7f223db42d..5287f1115f 100644
--- a/Documentation/git-send-email.adoc
+++ b/Documentation/git-send-email.adoc
@@ -389,7 +389,7 @@ Automating
 +
 Default is the value of `sendemail.suppressCc` configuration value; if
 that is unspecified, default to 'self' if --suppress-from is
-specified, as well as 'body' if --no-signed-off-cc is specified.
+specified, as well as 'body' if --no-signed-off-by-cc is specified.
 
 --[no-]suppress-from::
 	If this is set, do not add the From: address to the cc: list.
-- 
2.48.1
Patrick Steinhardt· Feb 18, 2025, 11:22 UTC · re: Matěj Cepl · lore

Re: [PATCH v2] doc(send-email): use correct name of the --signed-off-by-cc in git send-email doc

On Tue, Feb 18, 2025 at 09:58:38AM +0100, Matěj Cepl wrote:
Show 6 quoted lines
> I don’t think there is an option --signed-off-cc (without -by)
> for git send-email.
> 
> Signed-off-by: Matěj Cepl <mcepl@cepl.eu>
> ---
> Rebase on the top of the current master (Whoops!) and improve the commit message.
It's better, but still not quite there:
  - The subject now uses `doc(send-email)`, which is an idiom we don't
    use over here.
  - The commit message still doesn't use imperative mood.

I know, we're quite pedantic about commit messages over here, but please bear with us :) How about this instead:

    docs: fix misspelled `--no-signed-off-by-cc` in git-send-email(1)
    In git-send-email(1) we reference the `--no-signed-off-cc` command
    line argument. This argument does not exist and never has existed,
    but it is an obvious misspelling of `--no-signed-off-by-cc`.
    Fix the misspelled argument.
    Signed-off-by: ...

It would also be great if you send future revisions of a patch series in the same thread. You can do so via `--in-reply-to`, where the parameter is the message ID of the first message of the first version of your patch series.

Thanks!
Patrick
Matěj Cepl· Feb 18, 2025, 15:21 UTC · re: Patrick Steinhardt · lore

Re: [PATCH v2] doc(send-email): use correct name of the --signed-off-by-cc in git send-email doc

On Tue Feb 18, 2025 at 12:25 PM CET, Patrick Steinhardt wrote:
> I know, we're quite pedantic about commit messages over here, but please
> bear with us :) How about this instead:

No, I don’t think you are and I don’t think I will. There are no rules, there are people who are in and can do whatever they want to do and anybody who dares to contribute for the first time is needlessly harrassed. See this:

mitmanek:git (git-send-email-manpage-fix) $ git log --oneline --follow Documentation/git-send-email.adoc |head f18690968f doc(send-email): use correct name of the --signed-off-by-cc in git send-email doc 1f010d6bdf doc: use .adoc extension for AsciiDoc files 7ffcbafbf3 send-email: document --mailmap and associated configuration c038a6f1d7 send-email: teach git send-email option to translate aliases 0620ae0f5b doc: format alternatives in synopsis 82d75402d5 documentation: send-email: use camel case consistently 2162f9f6f8 doc: enforce dashes in placeholders d05b08cd52 doc: switch links to https 3ec6167567 send-email: handle to/cc/bcc from --compose message e0c7e2c326 doc/send-email: mention handling of "reply-to" with --compose cf6cac2005 documentation: wording improvements ba92106e93 send-email: add --header-cmd, --no-header-cmd options 839ebad442 send-email docs: Remove mention of discontinued gmail feature ba4324c4e1 e-mail workflow: Message-ID is spelled with ID in both capital letters a2634646eb docs: git-send-email: difference between ssl and tls smtp-encryption bac1d52cfe send-email docs: de-duplicate configuration sections a2ce608244 send-email docs: add format-patch options cd5b33fbdc git-send-email: add option to specify sendmail command 155067ab4f git-send-email.txt: mention less secure app access with Gmail 3abd4a67d9 Documentation: stylistically normalize references to Signed-off-by: mitmanek:git (git-send-email-manpage-fix) $

This is not being pedantic about anything. And all that in the email-driven project, where changing the patch before applying is absolutely no problem whatsoever. And all that harassment for three character change.

This harassment and complete NIH-syndrom ruling everything (git is the only project I have met so far, which didn’t settle on the vaguely K&R style for shell scripting, but why not to pester newbies with need to have specific settings in everything just for this one project, right?) leads me to abandoning this and couple of other attempts to contribute. Of course, accepting conventional commits (the only real standard in commit messages I know about) is beneath your station in life.

Have a nice day,
Matěj
-- 
http://matej.ceplovi.cz/blog/, @mcepl@en.osm.town
GPG Finger: 3C76 A027 CA45 AD70 98B5  BC1D 7920 5802 880B C9D8
 
Afraid to die alone?
Become a bus driver.
  -- alleged easter egg in notepad++
kristofferhaugsbakk@fastmail.com· Feb 5, 2026, 16:24 UTC · re: Matěj Cepl · lore

[PATCH v3] doc: send-email: correct --no-signed-off-by-cc misspelling

From: Matěj Cepl <mcepl@cepl.eu>

I don’t think there is an option --signed-off-cc (without -by) for git send-email.

Signed-off-by: Matěj Cepl <mcepl@cepl.eu>
[kh: rebased and changed subject to house style]
Signed-off-by: Kristoffer Haugsbakk <code@khaugsbakk.name>
---
Notes (series):
    v3:
    I left the rest of the commit message alone because I think it is okay.
 Documentation/git-send-email.adoc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to Documentation/git-send-email.adoc +1 −1
diff --git a/Documentation/git-send-email.adoc b/Documentation/git-send-email.adoc
index ebe8853e9f5..44ae2ed5bff 100644
--- a/Documentation/git-send-email.adoc
+++ b/Documentation/git-send-email.adoc
@@ -435,7 +435,7 @@ Automating
 +
 Default is the value of `sendemail.suppressCc` configuration value; if
 that is unspecified, default to `self` if `--suppress-from` is
-specified, as well as `body` if `--no-signed-off-cc` is specified.
+specified, as well as `body` if `--no-signed-off-by-cc` is specified.
 
 --suppress-from::
 --no-suppress-from::

base-commit: 67ad42147a7acc2af6074753ebd03d904476118f
-- 
2.53.0.26.g2afa8602a26

← back to recent threads