Thread (6 messages) flat view 6 messages, 4 authors, 2016-06-15

Re: [PATCH] Ignore end-of-line style when computing similarity score for rename detection

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:43:18

Possibly related (same subject, not in this thread)

Hi,

On Thu, 28 Jun 2007, Steven Grimm wrote:
Johannes Schindelin wrote:
quoted
Somehow I think that this should be triggered by "--ignore-space-at-eol",
_and_ be accompanied by a test case.
  
Should --ignore-space-at-eol be an option to git-merge? Merges are where 
this functionality matters; for simple diffs, --ignore-space-at-eol 
actually already covers it.
Good point. However, I fail to see how the similarity detection should be 
so decoupled from the application. IOW what good is it if two files are 
rated similar if the merge cannot handle the CRLF/LF differences properly?

So two points here: since the merges are what you target, you definitely 
should mention that in the commit message. And you should make sure that 
all this trickery only kicks in when the merge has a chance to succeed.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help