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

Re: git merge ignoring whitespace

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 5, 2009, 23:44 UTC
Message-ID
<7vtz78a1pn.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<2754.1233876289@relay.known.net>
layer <layer@known.net> writes:
Show 13 quoted lines
> Junio, on 3/25/07 you said:
>
>   I know Linus does not like applying patches with whitespace
>   fuzziness, but I personally think a merge that ignores
>   whitespace changes would be a useful thing to have, without
>   having to call an external specialized merge backend. In other
>   words, I do not think it is so special a thing to want to, but
>   it is rather an unfortunate norm.  I am not opposed to give
>   git-merge-recursive a command line option to tell it to merge
>   ignoring the whitespace-only changes, when it does the 3-way
>   file-level merge internally.
>
> Whatever came of this?
I do not know.  When I say "I am not opposed to", I mean just that. 

I may consider it a bad mode of operation, and I may hesitate to encourage people to use it as their default. But I do not feel strongly enough to veto it---as long as it is a rope you need to explicitly ask for to hang yourself with, I think it is Ok if we allowed such an option.

It is not my itch to scratch, and I do not keep track.
Did you submit a patch that implements the new feature cleanly?
Previous: layerNext: layer
Message 2 of 4 in “git merge ignoring whitespace”
  1. layerFeb 5, 2009
  2. Junio C HamanoFeb 5, 2009
  3. layerFeb 5, 2009
  4. Johannes SchindelinFeb 13, 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.