Don Slutz <Don.Slutz <at> SierraAtlantic.com> writes:

quoted text
> 
> This is a patch set to do the work from mail message:
> 
> http://kerneltrap.org/mailarchive/git/2007/8/7/254415
> 
> It is split into 6 parts.  This is because the tests do not work on
> Linux if autocrlf=true.
> 
> 1) The change to autocrlf=true by default for cygwin

I help maintain the cygwin packaging of git, and I don't particularly like 
patch 1/6.  Cygwin very much recommends that users stick with binary mounts, 
where cr's are not inserted/stripped by default, and enabling autocrlf in that 
situation is asking for problems (namely, the same set of problems as you would 
have on Linux with autocrlf=true).  While adding an environment variable is 
useful for the case of a user building git on cygwin with the intent of using a 
text-mode mount, it goes against the grain of cygwin's recommendation of not 
using text-mode mounts in the first place.  I would rather see the default 
remain false, even on cygwin, unless the person building git takes steps to 
activate it themselves.

-- 
Eric Blake



--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html