[PATCH] D54328: Added 'cd -' built in to lit

Joerg Sonnenberger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 13 15:36:32 PST 2018


joerg added a comment.

I would actually rule out subshells as well. I see no reason why a test case should have to use them, i.e. compared to just change the working directory back afterwards. As I said, I want to be able to repeat a problematic command manually and interactively as easy as possible. Using non-portable shell features is a problem for that. Depending on state between commands is also annoying. Let's really try to keep it minimal and where necessary, adjust test case guidelines and test cases to simplify things.


Repository:
  rL LLVM

https://reviews.llvm.org/D54328





More information about the llvm-commits mailing list