[Lldb-commits] [PATCH] D104413: Fixed use of -o and -k in LLDB under Windows when statically compiled with vcruntime.

Raphael Isemann via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 4 01:32:07 PDT 2021


teemperor accepted this revision.
teemperor added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: lldb/test/API/python_api/file_handle/TestFileHandle.py:920
+            self.assertTrue(re.search(r'Show a list of all debugger commands', output))
+            self.assertTrue(re.search(r'llvm revision', output))
----------------
Can you drop this line? Downstream tends to patch the version output with their own branding. Maybe `help command alias` and then check for `Define a custom command in terms of an existing command`.

Also I think you can use `self.assertIn('Show a list of all debugger commands', output)` here.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104413/new/

https://reviews.llvm.org/D104413



More information about the lldb-commits mailing list