[llvm] [lit] Ignore src permissions when copying lit_test_times (PR #121377)

Angus Lees via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 13 02:52:13 PST 2025


anguslees wrote:

> What do you think?

I think you're overcomplicating this, and I'm not sure why.

We already have a copy command that ignores the original permissions.  "Cat" and shell redirection are already used elsewhere so these  are not new dependencies.

This is an uncontroversial one line patch.  I agree this can be solved in other ways - and I don't object to any other solution.  But what am I missing here?

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


More information about the llvm-commits mailing list