This revision was automatically updated to reflect the committed changes.
Closed by commit rLLDB344623: [lldbsuite] Fix the mac version decorator to work
on non-mac platforms (authored by stella.stamenova, committed by ).
Repository:
rLLDB LLDB
https://reviews.llvm.org/D53208
Files:
packages/Python/lldbsuite/test/decorators.py
Index: packages/Python/lldbsuite/test/decorators.py
===================================================================
--- packages/Python/lldbsuite/test/decorators.py
+++ packages/Python/lldbsuite/test/decorators.py
@@ -192,10 +192,10 @@
py_version is None) or _check_expected_version(
py_version[0], py_version[1], sys.version_info)
skip_for_macos_version = (macos_version is None) or (
- _check_expected_version(
+ (platform.mac_ver()[0] != "") and (_check_expected_version(
macos_version[0],
macos_version[1],
- platform.mac_ver()[0]))
+ platform.mac_ver()[0])))
# For the test to be skipped, all specified (e.g. not None) parameters
must be True.
# An unspecified parameter means "any", so those are marked skip by
default. And we skip
Index: packages/Python/lldbsuite/test/decorators.py
===================================================================
--- packages/Python/lldbsuite/test/decorators.py
+++ packages/Python/lldbsuite/test/decorators.py
@@ -192,10 +192,10 @@
py_version is None) or _check_expected_version(
py_version[0], py_version[1], sys.version_info)
skip_for_macos_version = (macos_version is None) or (
- _check_expected_version(
+ (platform.mac_ver()[0] != "") and (_check_expected_version(
macos_version[0],
macos_version[1],
- platform.mac_ver()[0]))
+ platform.mac_ver()[0])))
# For the test to be skipped, all specified (e.g. not None) parameters must be True.
# An unspecified parameter means "any", so those are marked skip by default. And we skip
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
