]> git.ipfire.org Git - thirdparty/git.git/commit - sha1-file.c
Lazy man's auto-CRLF
authorLinus Torvalds <torvalds@linux-foundation.org>
Tue, 13 Feb 2007 19:07:23 +0000 (11:07 -0800)
committerJunio C Hamano <junkio@cox.net>
Wed, 14 Feb 2007 19:19:22 +0000 (11:19 -0800)
commit6c510bee2013022fbce52f4b0ec0cc593fc0cc48
tree38a06406362e3967060d2d0e1ba97ee620651145
parent437b1b20df4b356c9342dac8d38849f24ef44f27
Lazy man's auto-CRLF

It currently does NOT know about file attributes, so it does its
conversion purely based on content. Maybe that is more in the "git
philosophy" anyway, since content is king, but I think we should try to do
the file attributes to turn it off on demand.

Anyway, BY DEFAULT it is off regardless, because it requires a

[core]
AutoCRLF = true

in your config file to be enabled. We could make that the default for
Windows, of course, the same way we do some other things (filemode etc).

But you can actually enable it on UNIX, and it will cause:

 - "git update-index" will write blobs without CRLF
 - "git diff" will diff working tree files without CRLF
 - "git checkout" will write files to the working tree _with_ CRLF

and things work fine.

Funnily, it actually shows an odd file in git itself:

git clone -n git test-crlf
cd test-crlf
git config core.autocrlf true
git checkout
git diff

shows a diff for "Documentation/docbook-xsl.css". Why? Because we have
actually checked in that file *with* CRLF! So when "core.autocrlf" is
true, we'll always generate a *different* hash for it in the index,
because the index hash will be for the content _without_ CRLF.

Is this complete? I dunno. It seems to work for me. It doesn't use the
filename at all right now, and that's probably a deficiency (we could
certainly make the "is_binary()" heuristics also take standard filename
heuristics into account).

I don't pass in the filename at all for the "index_fd()" case
(git-update-index), so that would need to be passed around, but this
actually works fine.

NOTE NOTE NOTE! The "is_binary()" heuristics are totally made-up by yours
truly. I will not guarantee that they work at all reasonable. Caveat
emptor. But it _is_ simple, and it _is_ safe, since it's all off by
default.

The patch is pretty simple - the biggest part is the new "convert.c" file,
but even that is really just basic stuff that anybody can write in
"Teaching C 101" as a final project for their first class in programming.
Not to say that it's bug-free, of course - but at least we're not talking
about rocket surgery here.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Makefile
cache.h
config.c
convert.c [new file with mode: 0644]
diff.c
entry.c
environment.c
sha1_file.c