Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> writes: > On Sat, 14 Apr 2007, Junio C Hamano wrote: >> >> This adds an obviously sane pair of default attribute rules as built-ins. > > I'm not sure. > >> + "[attr]binary !diff !crlf", >> + "* diff crlf", > > Why would > > * diff crlf > > be "obviously sane"? > > In fact, I'd call it obviously insane. > > We do *not* want to default crlf to all files. We want the default to be > "automatic crlf depending on content". You do not have to worry. That's how "crlf" is defined. Paths you explicitly say !crlf will _not_ go through the existing core.autocrlf mechanism. "* crlf" just says, by default everybody is subject to core.autocrlf, and on sane platforms, core.autocrlf is by default off, hence you will not get LF <-> CRLF applied. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html