-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do you plan to add support for windows environment? #24
Comments
I haven't used Windows for more than 10 years, so I would not be able to, but I'm happy to accept patches. |
Actually it is not too bad since you added CLANG_FORMAT and CLANG_FORMAT_DIFF in the script.
1. Install clang-format at C:\Program Files\LLVM
2. set environment variable:
CLANG_FORMAT=C:\PROGRA~1\LLVM\bin\clang-format.exe
CLANG_FORMAT_DIFF=C:\PROGRA~1\LLVM\share\clang\clang-format-diff.py
1. Replace pre_commit with apply_format in repo/.git/hooks
Thanks!
Jin
From: Marco Barisione <[email protected]>
Sent: Thursday, January 2, 2020 10:11 AM
To: barisione/clang-format-hooks <[email protected]>
Cc: Yu, Jin <[email protected]>; Author <[email protected]>
Subject: {External} Re: [barisione/clang-format-hooks] Do you plan to add support for windows environment? (#24)
I haven't used Windows for more than 10 years, so I would not be able to, but I'm happy to accept patches.
What would you expect from a Windows port? Something that works on any Windows machine using standard Windows scripting?
How do people actually use git on Windows? Do they have a bash available?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#24?email_source=notifications&email_token=AOFGNYN7RVRQCY2K2DKVQHDQ3YU5DA5CNFSM4KBYRON2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH67LSA#issuecomment-570291656>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOFGNYIRWYCG26TEI7ZNWL3Q3YU5DANCNFSM4KBYRONQ>.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: