Enforce LF line endings in the git database.

Enforce LF line endings in the git database.

BUG=
R=ricow@google.com

Committed: https://github.com/dart-lang/sdk/commit/b1c1606e17f1302c67de06f6a88ac56d229ef334

[herhut, 2015-06-09 13:01:35]

herhut@google.com changed reviewers:
+ nweiz@google.com, ricow@google.com

[herhut, 2015-06-09 13:02:54]

We have quite some files that have CRLF endings. I propose to let git deal with
them and rewrite CRLF to LF on commit to the database.

The .gitattributes file will set this behavior for all source like things I
found.

[nweiz, 2015-06-09 19:11:50]

https://codereview.chromium.org/1168393003/diff/1/.gitattributes
File .gitattributes (right):

https://codereview.chromium.org/1168393003/diff/1/.gitattributes#newcode14
.gitattributes:14: *.bat -text
Is there actually a reason not to convert .bat files? I think they tend to be
edited on Linux and OS X, despite being run on Windows, and AFAIK nothing in
Windows actually chokes on LFs except for Notepad.

[ricow1, 2015-06-09 19:16:12]

https://codereview.chromium.org/1168393003/diff/1/tests/lib/mirrors/method_mi...
File tests/lib/mirrors/method_mirror_source_line_ending_crlf.dart (right):

https://codereview.chromium.org/1168393003/diff/1/tests/lib/mirrors/method_mi...
tests/lib/mirrors/method_mirror_source_line_ending_crlf.dart:5: // This file is
marked executable so source control will preserve the type of
This file is probably on purpose, please validate the rest

[herhut, 2015-06-24 07:42:30]

PTAL

Johnni tried for me on windows and it worked fine.

https://codereview.chromium.org/1168393003/diff/1/.gitattributes
File .gitattributes (right):

https://codereview.chromium.org/1168393003/diff/1/.gitattributes#newcode14
.gitattributes:14: *.bat -text
On 2015/06/09 19:11:50, nweiz wrote:
> Is there actually a reason not to convert .bat files? I think they tend to be
> edited on Linux and OS X, despite being run on Windows, and AFAIK nothing in
> Windows actually chokes on LFs except for Notepad.

No. Removed.

https://codereview.chromium.org/1168393003/diff/1/tests/lib/mirrors/method_mi...
File tests/lib/mirrors/method_mirror_source_line_ending_crlf.dart (right):

https://codereview.chromium.org/1168393003/diff/1/tests/lib/mirrors/method_mi...
tests/lib/mirrors/method_mirror_source_line_ending_crlf.dart:5: // This file is
marked executable so source control will preserve the type of
On 2015/06/09 19:16:12, ricow1 wrote:
> This file is probably on purpose, please validate the rest

Done.

[ricow1, 2015-06-24 07:44:35]

lgtm

[herhut, 2015-06-25 11:42:00]

Committed patchset #4 (id:30001) manually as
b1c1606e17f1302c67de06f6a88ac56d229ef334 (presubmit successful).