On 03/15/2015 09:02 PM, Robert Dailey wrote:
> I noticed that .gitattributes was not properly configured to utilize
> modern Git features:

Right, it was written back before Git had the 'text' and 'eol'
attributes.  Basically the strategy is:

* Use '-crlf' for files on which we want no conversion and make sure
  the repo blobs have the right newlines.
* Use 'crlf=input' for shell scripts to guarantee LF newlines.
* Let core.autocrlf decide everything else.  I personally prefer LF
  newlines on Windows, for example.

I wouldn't mind updating to use the modern capabilities but some of
our nightly testing machines may still have Git versions too old to
support 'text' and 'eol' so I'm hesitant to change the above.
I don't know when I'd have time to check them all.

> 3. General cleanup and removal of redundant attributes

Note that *.png is in .gitattributes because in some cases Git
fails to detect that they are binary and still converts the newlines
in the image header text.  I'm not aware of anything else in
the current .gitattributes that is not needed or is redundant.

> 1. Some files were not properly being handled by git

What files?  Perhaps we can at least get those corrected.

FYI, Utilities/cmbzip2/*.dsp are not used and could be deleted.

> 2. Git config settings on a per-machine basis affected the EOL
> normalization in the repository

We have server-side repository checks to prevent CRLF newlines
from appearing in files not marked as binary with -crlf.  Still,
this could be problematic for users working across platforms
locally.

Thanks,
-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to