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

Re: [PATCH] builtin-merge: fix a typo in an error message

From
Allan Caffee <allan.caffee@gmail.com>
Date
Apr 13, 2009, 21:41 UTC
Message-ID
<b2e43f8f0904131441j4d808431mc2c87531221c8f4d@mail.gmail.com>
In-Reply-To
<20090413181008.GA8273@linux.vnet>
On Mon, Apr 13, 2009 at 2:10 PM, Allan Caffee <allan.caffee@gmail.com> wrote:
Show 16 quoted lines
>
> Signed-off-by: Allan Caffee <allan.caffee@gmail.com>
> ---
>  builtin-merge.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/builtin-merge.c b/builtin-merge.c
> index 6a51823..0b58e5e 100644
> --- a/builtin-merge.c
> +++ b/builtin-merge.c
> @@ -764,7 +764,7 @@ static int suggest_conflicts(void)
>
>        fp = fopen(git_path("MERGE_MSG"), "a");
>        if (!fp)
> -               die("Could open %s for writing", git_path("MERGE_MSG"));
> +               die("Could not open %s for writing", git_path("MERGE_MSG"));
On second thought this commit message should probably read something more like:
builtin-merge.c: correct error message for failed open
Since the original message is actually wrong.
Previous: Allan CaffeeNext: Miklos Vajna
Message 2 of 5 in “builtin-merge: fix a typo in an error message”
  1. builtin-merge: fix a typo in an error messageAllan Caffee, Apr 13, 2009
  2. Allan CaffeeApr 13, 2009
  3. Miklos VajnaApr 13, 2009
  4. Allan CaffeeApr 27, 2009
  5. Junio C HamanoApr 27, 2009

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.