[Python-Dev] [Python-checkins] r42929 - python/trunk/Tools/scripts/svneol.py

[Thomas Heller]

Tim Peters wrote:
> [Tim Peters]
>>> Added:
>>>    python/trunk/Tools/scripts/svneol.py   (contents, props changed)
>>> Log:
>>> Simple utility to add svn:eol-style to text files under
>>> SVN control.  Like reindent.py, I expect to run this
>>> mindlessly from time to time, checking in whatever it
>>> happens to do ;-)
> 
> [Thomas Heller]
>> Should 'sln' and 'vcproj' be added to the extensions list?  I think these are
>> text-files too.  Although PCBuild\pcbuild.sln has a binary mime-type property,
>> so the script would not change that.
> 
> I don't know whether they're text files in the SVN eol-style "native"
> sense.  The corresponding file types under VC 6 were not:  in SVN
> terms, they should have svn:eol-style set to CRLF (Visual Studio
> required \r\n line ends for some inscrutable reason -- it would not

I know. In CVS, they had to be added using the -kb flag, IIRC.

> tolerate \n line ends, and this mattered when, e.g., some Linux-head
> tried to run Visual Studio from a Linux-native checkout of Python;
> CRLF was still required, and we "faked that" under CVS by calling
> those files binary; SVN offers finer control; I don't know how picky
> the .NET 2003 MSDev is about this).

I remember that someone has tested that the .vcproj files actually did work
even if they had the wrong line endings, so it should be ok.

AFAIK, noone tried that with the .sln file - anyway, I have removed the
svn:mem-type property and the script then set svn:eol-style native.
Maybe a different set of svn properties were better, I have to learn
about them.  The problem that I saw is that 'svn diff' refused to provide
a diff when pcbuild.sln had changed - I would consider this diff useful, both
in commit-emails as well as run on the command line before checking in.

The Linux-head is the release manager, prepaing the release on
a unixish platform, in combination with the windows guy who downloads
the source distribution and tries to compile that on Windows.

Thomas