[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