<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Dec 7, 2014 at 11:58 PM, Pawel Jasinski <span dir="ltr"><<a href="mailto:pawel.jasinski@gmail.com" target="_blank">pawel.jasinski@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sun, Dec 7, 2014 at 9:11 PM, Jeff Hardy <<a href="mailto:jdhardy@gmail.com">jdhardy@gmail.com</a>> wrote:<br>
> On Sun, Dec 7, 2014 at 12:04 PM, Pawel Jasinski <<a href="mailto:pawel.jasinski@gmail.com">pawel.jasinski@gmail.com</a>><br>
> wrote:<br>
>><br>
>> hi,<br>
>><br>
>> time.cs and nt.cs have mixed line ending. This is causing unnecessary<br>
>> pain when rebasing and produces ugly ^M in diff.<br>
>><br>
>> Would it make sense to fix it now?<br>
><br>
><br>
> Yes, please! (I noticed nt.cs the other night as well.)<br>
<br>
</div></div>I have checked IronLanguages main and ipy-2.7-maint<br>
Both have files with LF, CRLF and mixed. I don't want to touch the LF and CRLF.<br>
However the mixed files are recent introduction and pain to deal with.<br>
I will fix it with single commit and will not do the history rewriting<br>
unless someone give me good reason.<br>
<br>
I also checked ironpython3 and there are no mixed files.<br>
As a side effect of the check I did CRLF stats. I see most of the<br>
files committed with LF and 11 files with CRLF.<br>
Is this something to be fixed, nice to have or don't care?<br></blockquote><div><br></div><div>Ideally all files are LF, since those are handled everywhere (except Notepad) properly. However, it's possible those files are intentionally CRLF for some reason so I wouldn't blindly change them. I did that conversion on a Mac, so that's probably why they're consistent. :)</div><div><br></div><div>It's possible to configure autocrlf, but I don't know what thje proper setting should be.</div><div><br></div><div>Mixed files should not be there, though.</div><div><br></div><div>- Jeff</div></div></div></div>