Don Slutz <Don.Slutz <at> SierraAtlantic.com> writes: > > 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html