Author: Jonas Devlieghere Date: 2020-01-23T21:46:50-08:00 New Revision: be2bc6b1d0f60b45e3fb40c14ee7bd1b4f2fa88c
URL: https://github.com/llvm/llvm-project/commit/be2bc6b1d0f60b45e3fb40c14ee7bd1b4f2fa88c DIFF: https://github.com/llvm/llvm-project/commit/be2bc6b1d0f60b45e3fb40c14ee7bd1b4f2fa88c.diff LOG: [lldb/Util] Remove reproducers after replay Except for debugging purposes there's no point in leaving the reproducer behind on disk after replay. This patch adds a cleanup in the replay case. Added: Modified: lldb/utils/lldb-repro/lldb-repro.py Removed: ################################################################################ diff --git a/lldb/utils/lldb-repro/lldb-repro.py b/lldb/utils/lldb-repro/lldb-repro.py index 313d3855a025..f7bcd3baa7a4 100755 --- a/lldb/utils/lldb-repro/lldb-repro.py +++ b/lldb/utils/lldb-repro/lldb-repro.py @@ -43,8 +43,10 @@ def main(): # Create a new lldb invocation with capture or replay enabled. lldb = os.path.join(os.path.dirname(sys.argv[0]), 'lldb') new_args = [lldb] + cleanup = False if sys.argv[1] == "replay": new_args.extend(['--replay', reproducer_path]) + cleanup = True elif sys.argv[1] == "capture": new_args.extend([ '--capture', '--capture-path', reproducer_path, @@ -55,7 +57,10 @@ def main(): help() return 1 - return subprocess.call(new_args) + exit_code = subprocess.call(new_args) + if cleanup: + shutil.rmtree(reproducer_path, True) + return exit_code if __name__ == '__main__': _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
