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

Re: commiting while the current version is in conflict

From
Richard Hartmann <richih.mailinglist@gmail.com>
Date
Oct 17, 2008, 09:32 UTC
Message-ID
<2d460de70810170232o24fcd1d4pd1c7a7503e17b02@mail.gmail.com>
In-Reply-To
<7vwsg7mxb6.fsf@gitster.siamese.dyndns.org>
On Fri, Oct 17, 2008 at 10:37, Junio C Hamano <gitster@pobox.com> wrote:
> Eh, Sorry about that --- I did not mean "ignorant" in that sense.  Perhaps
> replacing the word with "unfortunate" would sound nicer?

A lot yes. No harm done, apologies for picking the bad interpretation of ignorant.

Show 5 quoted lines
> Don't you realize that is what the hook is doing already?  After making
> such an error, the definitive information is lost, because the user told
> the tool that the resolution is done and the file is ready to be
> committed) by mistake.  Even then the hook is trying its best to help the
> user.

You misunderstood me, there. I was thinking about the pre-add hook while writing the above. Doing anything at commit time is obviously wrong.

> As to pre-add hook, I am not enthused, but if somebody sends in a clean
> patch, I wouldn't be opposed to it at least in principle.

An implicit need for it does apparently exist. Else, the default hook for pre-commit would not need to check for conflicts.

Richard
Previous: Junio C HamanoNext: Jakub Narebski
Message 4 of 7 in “Re: commiting while the current version is in conflict”
  1. Junio HamanoOct 16, 2008
  2. Richard HartmannOct 17, 2008
  3. Junio C HamanoOct 17, 2008
  4. Richard HartmannOct 17, 2008
  5. Jakub NarebskiOct 17, 2008
  6. Richard HartmannOct 17, 2008
  7. Junio C HamanoOct 17, 2008

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.