[libcxx-commits] [libcxx] [libcxx] Require non-root-user for tests that check permissions (PR #119940)

Nikolas Klauser via libcxx-commits libcxx-commits at lists.llvm.org
Sat Dec 14 04:18:33 PST 2024


https://github.com/philnik777 requested changes to this pull request.

This patch doesn't ever enable `non-root-user`, so this is just dead code. I also don't really see the point. It's pretty trivial to change the user, at least in Docker. Another problem is that this will just hide that some tests don't run instead of informing the user.

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


More information about the libcxx-commits mailing list