[dmd-internals] LF and CRLF on git

Brad Roberts braddr at puremagic.com
Tue Jan 15 19:31:19 PST 2013


On 1/15/2013 7:21 PM, Alex Rønne Petersen wrote:
> On Wed, Jan 16, 2013 at 4:19 AM, Andrei Alexandrescu <andrei at erdani.com> wrote:
>> On 1/15/13 10:14 PM, Walter Bright wrote:
>>>
>>> I also like the idea of the autotester rejecting any files with CRLF.
>>> Brad?
>>
>>
>> Before that. It's a pre-commit or post-receive matter. Brad? :o)
> 
> You cannot put arbitrary Git hooks into a repository that will be run
> on end users' machines out of the box.
> 
> (This would be a huge security hole in Git.)

Correct, and github has no mechanism for it either.  So, I'm happy to add this checking to the load balancer, but
someone else build the hook and play with it and get it working w/in their own environment first .  There's no reason it
shouldn't be put in developer repositories and catch the problem BEFORE they get committed locally.  The auto tester can
be the second layer of defense.

Later,
Brad


More information about the dmd-internals mailing list