[PATCH] D78846: Fix .arclint on Windows

Scott Linder via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 11:17:48 PDT 2020


scott.linder accepted this revision.
scott.linder added a comment.
This revision is now accepted and ready to land.

I don't have a way to test this, but it seems like an OK change to me, modulo the fact that I don't know the effect of the weirdness @Meinersbur brings up.

Does git really depend on bash, or just "POSIX sh"? We could instead change the script to invoke /bin/sh and audit it to be POSIX.

I think we should really just rewrite it in Python, which I intended to do but haven't had a chance to yet.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78846/new/

https://reviews.llvm.org/D78846





More information about the llvm-commits mailing list