Re: [Lldb-commits] [PATCH] D44526: [dotest] Clean up test folder clean-up

2018-03-20 Thread Jim Ingham via lldb-commits
> On Mar 20, 2018, at 4:55 AM, Pavel Labath via lldb-commits > wrote: > > Deleting the test build dir is fairly easy. I can whip up a patch for that, > but I'm not sure if that's the part that is bothering you the most here. > Dealing with the log files is a bot more complicated and there do

Re: [Lldb-commits] [PATCH] D44526: [dotest] Clean up test folder clean-up

2018-03-20 Thread Pavel Labath via lldb-commits
Deleting the test build dir is fairly easy. I can whip up a patch for that, but I'm not sure if that's the part that is bothering you the most here. Dealing with the log files is a bot more complicated and there doesn't seem to be a clear consensus on what to do with them. The last discussion abou

Re: [Lldb-commits] [PATCH] D44526: [dotest] Clean up test folder clean-up

2018-03-19 Thread Greg Clayton via lldb-commits
I would be nice to have the option to nuke each test build directory if the test passed. Can be an option that we specify. That way, the only folders left over could be the tests that are failing. The options doesn't need to default to true, but if anyone is in that code and would know how to do