owenca wrote:
See #75268.
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1 @@
+py -3 git-clang-format %*
owenca wrote:
@llvm-beanz This doesn't seem to work if `git-clang-format` is run from another
directory:
```
C:\Users\Owen\llvm-project>clang\tools\clang-format\git-clang-format
C:\Users\Owen\llvm-project>py -3
https://github.com/llvm-beanz closed
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1 @@
+py -3 git-clang-format %*
llvm-beanz wrote:
`py.exe` is the Windows python launcher. It comes with Python for Windows (not
msys2 or cygwin where the shebang line would work). It is the Windows way of
choosing which python version you want. You
HazardyKnusperkeks wrote:
I have no real opinion on this matter, as I don't use that program/script. But
when it helps the windows users, why not? I also see no harm.
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
@@ -0,0 +1 @@
+py -3 git-clang-format %*
HazardyKnusperkeks wrote:
What is `py`? I have no program called `py` on my machine (Windows with msys2),
so you would possibly break that? Why not write `python` or `python3`?
owenca wrote:
@mydeveloperday what do you think?
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-beanz wrote:
> Can you open an issue and link to it from here?
Filed #69643.
> It seems `git clang-format` works in Command Prompt without this new batch
> file.
IIUC, `git clang-format` works if you have WSL installed, but not with a
default Windows installation. We definitely have
https://github.com/llvm-beanz edited
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
> This allows git-clang-format to be used on a Windows terminal without
> manually needing to find the path and invoke the python interpreter. We have
> a similar script for `scan-build`.
Can you open an issue and link to it from here? It seems `git clang-format`
works in
https://github.com/damyanp approved this pull request.
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Chris B (llvm-beanz)
Changes
This allows git-clang-format to be used on a Windows terminal without manually
needing to find the path and invoke the python interpreter. We have a similar
script for `scan-build`.
---
Full diff:
https://github.com/llvm-beanz created
https://github.com/llvm/llvm-project/pull/69228
This allows git-clang-format to be used on a Windows terminal without manually
needing to find the path and invoke the python interpreter. We have a similar
script for `scan-build`.
>From
14 matches
Mail list logo
