E.g. the mingw python distributed in msys2 (the mingw one, which is a normal win32 application and doesn't use the msys2 runtime itself), despite being a normal win32 python, still uses forward slashes. This works fine for other cases, but when constructing a raw NT path, all path separators must be backslashes.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Admittedly, this python doesn't really pass all tests anyway, as some tests rely on path formatting to match between different components - but it does help running some number of tests with it at least.
Comment Actions
lgtm
llvm/utils/lit/lit/util.py | ||
---|---|---|
157 | I feel like path.replace('/', '\\') would be the more obvious spelling (fewer temporary strings), but they are functionally equivalent. |
llvm/utils/lit/lit/util.py | ||
---|---|---|
157 | Ah, yes, will change it to that form before pushing the change. Thanks! |
I feel like path.replace('/', '\\') would be the more obvious spelling (fewer temporary strings), but they are functionally equivalent.