michaelplatings added inline comments.
================ Comment at: llvm/utils/lit/lit/llvm/config.py:171 + # in the tests that do the same thing. + (_, try_touch_err) = self.get_process_output(["touch", "-a", "-t", "199505050555.55", f.name]) + if try_touch_err != "": ---------------- lenary wrote: > michaelplatings wrote: > > It looks like this command will be run on Windows. I think it will fail and > > cause False to be returned, which is the desired result, but this appears > > to be by accident rather than design. Therefore I'm inclined to agree with > > @int3 that a hard-coded check would be preferable. > I am going to add the hardcoded checks, but I think `touch` is available in > windows, it should be in the same directory as all the git binaries. This is definitely tangential to the change, but in case it's useful to know: conventionally only `C:\Program Files\Git\bin` is added to the path on Windows, not `C:\Program Files\Git\usr\bin`. `C:\Program Files\Git\bin` only contains `bash.exe`, `git.exe` and `sh.exe`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144638/new/ https://reviews.llvm.org/D144638 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
