[PATCH] doc: fix typo

[PATCH] doc: fix typo

[Matěj Cepl]

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>
---
 Documentation/git-send-email.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index bc3ef45acb..d762058907 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -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

Re: [PATCH] doc: fix typo

[Patrick Steinhardt]

On Mon, Feb 17, 2025 at 02:08:50PM +0100, Matěj Cepl wrote:
> I don’t think there is an option --signed-off-cc (without -by)
> for git send-email.

A couple comments regarding the commit subject and body:

  - The subject should point out _where_ you're fixing a typo.

  - The message should be assertive and use imperative moot, as if
    instructing the documentation to change.

I'd recommend to have a look at commit messages that relate to your file
by doing `git log --follow Documentation/git-send-email.adoc` to get a
bit of an idea of how this could look like.

> Signed-off-by: Matěj Cepl <mcepl@cepl.eu>
> ---
>  Documentation/git-send-email.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

You should probably rebase your commit on the latest `master` as the
file has since been renamed to "git-send-email.adoc".

> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
> index bc3ef45acb..d762058907 100644
> --- a/Documentation/git-send-email.txt
> +++ b/Documentation/git-send-email.txt
> @@ -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.

The change itself looks obviously correct to me, thanks!

Patrick

Re: [PATCH] doc: fix typo

[Junio C Hamano]

Patrick Steinhardt <ps@pks.im> writes:

> You should probably rebase your commit on the latest `master` as the
> file has since been renamed to "git-send-email.adoc".

This looks like a 'maint' worthy material, though.

Downporting with cherry-pick should be able to deal with the rename,
but queuing with the old name on an older maintenance track and
upmerging should be cleaner, so .txt is fine in this case.

Not that I plan to do Git 2.48.2 anytime soon, so either way it does
not really matter, though ;-)

Thanks for a review.


>> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
>> index bc3ef45acb..d762058907 100644
>> --- a/Documentation/git-send-email.txt
>> +++ b/Documentation/git-send-email.txt
>> @@ -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.
>
> The change itself looks obviously correct to me, thanks!
>
> Patrick

Re: [PATCH] doc: fix typo

[Matěj Cepl]

<https://lore.kernel.org/git/20250218085913.1381537-1-mcepl@cepl.eu/>
and
<https://lore.kernel.org/git/D7VO9H4A9DMN.JK0CKJNDX3XZ@cepl.eu/>

[PATCH] doc: fix typo

[Aristo Chen]

Fix typo from "to" to "do"

Signed-off-by: Aristo Chen <aristo.chen@canonical.com>
---
 doc/usage/blkmap.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/usage/blkmap.rst b/doc/usage/blkmap.rst
index 75f736c259f..4e9091e8ffa 100644
--- a/doc/usage/blkmap.rst
+++ b/doc/usage/blkmap.rst
@@ -39,7 +39,7 @@ network. Getting the image to the target is easy enough:
 
    dhcp ${ramdisk_addr_r} rootfs.ext4
 
-But now we are faced with a predicament: how to we extract the kernel
+But now we are faced with a predicament: how do we extract the kernel
 image? Block maps to the rescue!
 
 We start by creating a new device:
-- 
2.34.1

Re: [PATCH] doc: fix typo

[Quentin Schulz]

Hi Aristo,

On 4/24/25 11:14 AM, Aristo Chen wrote:
> Fix typo from "to" to "do"
> 
> Signed-off-by: Aristo Chen <aristo.chen@canonical.com>

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin