[cfe-dev] git-clang-format

Zachary Turner zturner at google.com
Fri Feb 6 23:03:05 PST 2015


Ignore this message.  Apparently I shouldn't be trying to do work at 11
o'clock.  :]  It actually works fine without being a .py file.  Python just
wasn't in my path, so it was giving me a misleading error message.

On Fri Feb 06 2015 at 10:37:46 PM Chandler Carruth <chandlerc at google.com>
wrote:

>
> On Fri, Feb 6, 2015 at 9:53 PM, Zachary Turner <zturner at google.com> wrote:
>
>> Hi Daniel,
>>
>> Is there any specific reason why
>> clang/tools/clang-format/git-clang-format is a shell script
>> with #!/usr/bin/env python as opposed to being named git-clang-format.py?
>>
>> With the former, it doesn't work on windows, whereas with the latter it
>> does.  I renamed the file to git-clang-format.py and tested it out on
>> Windows, and it works great.  So if there's no objection I think we should
>> rename the file.
>>
>> Thoughts?
>>
>
> I don't have any specific problems with this plan, but one related note --
> we should do something to pin it to Python 2.7 or update it to work with
> Python 3.0. I've had to manually hack around this a bunch.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20150207/ad3224ae/attachment.html>


More information about the cfe-dev mailing list