[lldb/test] Clean up version checking.
authorJordan Rupprecht <rupprecht@google.com>
Mon, 21 Sep 2020 23:19:28 +0000 (16:19 -0700)
committerJordan Rupprecht <rupprecht@google.com>
Mon, 21 Sep 2020 23:19:28 +0000 (16:19 -0700)
commit307b7a1d665898d0e980461919996b6a670a4847
tree23de3d12e835e9a1b0439ae8fef1abff596374dd
parentca8321574d62cdccc85cf222ae43e0b48c3dfe96
[lldb/test] Clean up version checking.

A few fixes while trying to figure out why tests are being skipped for arsenm:

- We check `$compiler -v`, but `-v` is `--verbose`, not `--version`. Use the long flag name.
- We check all lines matching `version ...`, but we should exit early for the first version string we see (which should be the main one). I'm not sure if this is the issue, but perhaps this is causing some users to skip some tests if another "version ..." is showing up later.
- Having `\.` in a python string is triggering pylint warnings, because it should be escaped as a regex string, e.g. `r'\.' However, `.` in a character class does not need to be escaped, as it matches only a literal `.` in that context.

Reviewed By: JDevlieghere

Differential Revision: https://reviews.llvm.org/D88051
lldb/packages/Python/lldbsuite/test/lldbtest.py