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

Re: Is XDL_MERGE_ZEALOUS too zealous (or maybe not zealous enough)?

From
Matt McCutchen <matt@mattmccutchen.net>
Date
Oct 20, 2008, 03:42 UTC
Message-ID
<1224474120.19785.3.camel@mattlaptop2.local>
In-Reply-To
<7v3ais5hb3.fsf_-_@gitster.siamese.dyndns.org>
On Sun, 2008-10-19 at 15:52 -0700, Junio C Hamano wrote:
Show 26 quoted lines
> However, the usual simplified merge shows this (run "git checkout --merge
> builtin-checkout.c" if you have done the above):
> 
> <<<<<<< ours
> 	/* --track without -b should DWIM */
> 	if (0 < opts.track && !opts.new_branch) {
> 		const char *argv0 = argv[0];
> 	...
> 		opts.new_branch = argv0 + 1;
> 	}
> 
> 	if (opts.track == BRANCH_TRACK_UNSPECIFIED)
> 		opts.track = git_branch_track;
> =======
> 	if (conflict_style) {
> 		opts.merge = 1; /* implied */
> 		git_xmerge_config("merge.conflictstyle", conflict_style, NULL);
> 	}
> 
> 	if (!opts.new_branch && (opts.track != git_branch_track))
> 		die("git checkout: --track and --no-track require -b");
> >>>>>>> theirs
> 
> Removing the two lines from the simplified "theirs" is not what I would
> suggest (it would be actively wrong), but I wonder if we can do something
> clever to help users with a merge like this.

IMHO, the solution is just to use diff3 style. I never understood how I was supposed to intuit the correct result of a merge from the two sides without seeing the common ancestor, so I am glad to have the diff3 style working now.

Matt
Previous: Junio C HamanoNext: Johannes Schindelin
Message 4 of 5 in “"git checkout: --track and --no-track require -b" check accidentally resurrected?”
  1. Matt McCutchenOct 19, 2008
  2. Junio C HamanoOct 19, 2008
  3. Is XDL_MERGE_ZEALOUS too zealous (or maybe not zealous enough)?Junio C Hamano, Oct 19, 2008
  4. Matt McCutchenOct 20, 2008
  5. Johannes SchindelinOct 20, 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.