labath added inline comments.
================ Comment at: lldb/packages/Python/lldbsuite/test/decorators.py:861 + def is_reproducer(): + if ('LLDB_REPRODUCER_CAPTURE_PATH' in os.environ or 'LLDB_REPRODUCER_REPLAY_PATH' in os.environ): + return "reproducers unsupported" ---------------- Should this use variables from configuration.py now? ================ Comment at: lldb/test/API/lit.cfg.py:68 +if lldb_repro_mode: + lit_config.note("Running Shell test with lldb-repo in {} mode.".format(lldb_repro_mode)) + if lldb_repro_mode == 'capture': ---------------- lldb-repro ================ Comment at: lldb/test/API/lldbtest.py:89 + reproducer_path = os.path.join(tempfile.gettempdir(), testFile) + if 'lldb-repro-capture' in test.config.available_features: ---------------- I remember some objections to shoving everything in $TMP. Are we not able to put this in the build dir somewhere? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77588/new/ https://reviews.llvm.org/D77588 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits