[llvm] [llvm] Add support for running tests as root (PR #75285)

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 3 13:39:52 PST 2024


tstellar wrote:

Thanks for the feedback.  I would prefer to keep the patch as-is.  The reason I don't want to use os.getuid() is that it is only available on Unix systems, and I don't want to have to add extra code to do OS detection.  I also don't want to do a specific ACL test, because creating temporary files is adding another point of failure to lit start up.  Since this check is only needed by 4 tests, I want to make it as simple as possible and reduce the risk that it will impact users running as non-root, which is the most common case.

https://github.com/llvm/llvm-project/pull/75285


More information about the llvm-commits mailing list