Re: [PATCH/RFC v2 1/4] Add "core.eolStyle" variable to control end-of-line conversion

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Dmitry Potapov
Date: Monday, May 10, 2010 - 1:13 am

On Sun, May 09, 2010 at 10:09:37PM +0200, Finn Arne Gangstad wrote:

First of autocrlf is safe as it is implemented now. Second, to do
something to get a new with CRLF into the repo is really stupid.
The whole point of autocrlf is being automatic and do not have the
user to worry about CRLF when he adds a new file.

The only real problem I am aware of is that some repository are not
compatible with autocrlf conversion, because they store text files
with different endings and do not have appropriate .gitatributes to
describle what text files should and should not be converted. So, each
user has to disable autocrlf in them manually and then re-checkout all
files using (rm -rf * && git checkout -f), which is confusing for many
users. In fact, you do not have to disable autocrlf, you can add a few
lines to .git/info/attributes to make it autocrlf compatible, but again
many users even not aware about this file, let alone what needs to be
added. Further, the problem amplified by the fact that you have to do
the same procedure every time when you do cloning, and though cloning
is not most common operation, it happens often enough to annoy many
users.

I believe that the right solution is to be able to enable autocrlf but
only for those repositories that are marked as autocrlf compatible by
upstream.


Dmitry
--
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
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH/RFC v2 0/4] End-of-line normalization, take 2 (now ..., Eyvind Bernhardsen, (Sat May 8, 2:46 pm)
[PATCH/RFC v2 2/4] Add tests for per-repository eol normal ..., Eyvind Bernhardsen, (Sat May 8, 2:46 pm)
[PATCH/RFC v2 3/4] Pass eol conv mode as an argument inste ..., Eyvind Bernhardsen, (Sat May 8, 2:46 pm)
[PATCH/RFC v2 4/4] Add per-repository eol normalization, Eyvind Bernhardsen, (Sat May 8, 2:46 pm)
Re: [PATCH/RFC v2 1/4] Add "core.eolStyle" variable to con ..., Dmitry Potapov, (Mon May 10, 1:13 am)