================ @@ -1794,8 +1803,36 @@ def SymbolicateCrashLogs(debugger, command_args, result, is_command): result.SetError(str(e)) return + # To avoid breaking existing users, we should keep supporting legacy flags + # even if we don't use them / advertise them anymore. + if options.interactive: + options.mode = CrashLogLoadingMode.interactive + elif options.batch: + options.mode = CrashLogLoadingMode.batch + + if ( + options.mode ---------------- bulbazord wrote:
Isn't `options.mode` always set to interactive? It defaults to that up in the argparse code, so is this check necessary? https://github.com/llvm/llvm-project/pull/94575 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits