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

Re: [PATCH v3 0/2] git-gui: change to display the combined diff in the case of conflicts.

From
Johannes Sixt <j.sixt@viscovery.net>
Date
Mar 31, 2010, 13:51 UTC
Message-ID
<4BB3534E.3080803@viscovery.net>
In-Reply-To
<t2u2cfc40321003310523u8fda9baeo883d2e0b3c6fa807@mail.gmail.com>
Am 3/31/2010 14:23, schrieb Jon Seymour:
> What about the safe (but complicated) diff -c by default (to prevent
> misinformed Use Remote/Local decisions, by default) and a "Conflicts
> Only" option (disabled by default) that shows the diff --cc output for
> those who know what they are doing?
You would like to have the worst of all three?
1. dangerous menu options
2. incomprehensible diff -c output
3. and a config option on top of it
I think we should have none of them.

Yes, I suggested diff -c, but after seeing how it works (better: not works), I changed my mind.

What sort of user guidance is it to present alluring menu options "Use local/remote version", but at the same time a scaring amount of diff -c output.

-- Hannes
Previous: Jon SeymourNext: Jon Seymour
Message 7 of 10 in “git-gui: change to display the combined diff in the case of conflicts.”
  1. 0/2 git-gui: change to display the combined diff in the case of conflicts.Jon Seymour, Mar 30, 2010
  2. Johannes SixtMar 31, 2010
  3. Jon SeymourMar 31, 2010
  4. Johannes SixtMar 31, 2010
  5. Jon SeymourMar 31, 2010
  6. Jon SeymourMar 31, 2010
  7. Johannes SixtMar 31, 2010
  8. Jon SeymourMar 31, 2010
  9. Johannes SixtApr 2, 2010
  10. Jon SeymourApr 4, 2010

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.