git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Fix word "does" sticking to "not" by adding space

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 20, 2022, 00:28 UTC
Message-ID
<xmqqh6xq2a1g.fsf@gitster.g>
In-Reply-To
<pull.1393.git.git.1671416810927.gitgitgadget@gmail.com>
"Sheepolution via GitGitGadget" <gitgitgadget@gmail.com> writes:
> Subject: Re: [PATCH] Fix word "does" sticking to "not" by adding space

Not incorrect per-se, but the above fails to say a more important thing---where does the typo appear?

    Subject: [PATCH] mingw: fix typo in an error message from ownership check
or something, perhaps.
> From: =?UTF-8?q?Dani=C3=ABl=20Haazen?= <danielhaazen@hotmail.com>
Here is where the proposed log message goes, not ...
> Signed-off-by: Daniël Haazen <danielhaazen@hotmail.com>
> ---
>     git: fix word "does" sticking to "not" in message
Show 7 quoted lines
>     
>     When a repository is on a FAT32 file system it will send a message to
>     the user explaining they should use git config --global --add
>     safe.directory. To prevent a long line in the file the message is made
>     up of two string literals. In this change the first literal has a space
>     added to it to fix the issue that the last and the first word of the two
>     literals stick together like the example image below.
... here.
     When a repository is on a FAT32 file system it will send a
     message that the path ownership cannot be determined.  Fix a
     typo in the message.
should be sufficient.
Show 11 quoted lines
> diff --git a/compat/mingw.c b/compat/mingw.c
> index d614f156df1..af397e68a1d 100644
> --- a/compat/mingw.c
> +++ b/compat/mingw.c
> @@ -2752,7 +2752,7 @@ int is_path_owned_by_current_sid(const char *path, struct strbuf *report)
>  			/*
>  			 * On FAT32 volumes, ownership is not actually recorded.
>  			 */
> -			strbuf_addf(report, "'%s' is on a file system that does"
> +			strbuf_addf(report, "'%s' is on a file system that does "
>  				    "not record ownership\n", path);
Thanks for spotting the mistake.  Very much appreciated.
Previous: Sheepolution via GitGitGadget
Message 2 of 2 in “Fix word "does" sticking to "not" by adding space”
  1. Fix word "does" sticking to "not" by adding spaceSheepolution via GitGitGadget, Dec 19, 2022
  2. Junio C HamanoDec 20, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.