summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZach White <skullydazed@gmail.com>2020-12-18 16:42:30 -0800
committerDrashna Jael're <drashna@live.com>2021-01-12 22:46:09 -0800
commit336e00731044ad0fb28cae1ce63929b2244798fb (patch)
treecaff408c1d9d5eab47fccadd563812b6c4e77722
parent18b03049f39e0993fb1d8a58415b53aa588f84b0 (diff)
simplify qmk doctor to make room for #11208 (#11242)
-rwxr-xr-xlib/python/qmk/cli/doctor.py30
1 files changed, 17 insertions, 13 deletions
diff --git a/lib/python/qmk/cli/doctor.py b/lib/python/qmk/cli/doctor.py
index 4fe318b636..25e2e239cb 100755
--- a/lib/python/qmk/cli/doctor.py
+++ b/lib/python/qmk/cli/doctor.py
@@ -281,6 +281,22 @@ def is_executable(command):
return False
+def os_tests():
+ """Determine our OS and run platform specific tests
+ """
+ platform_id = platform.platform().lower()
+
+ if 'darwin' in platform_id or 'macos' in platform_id:
+ return os_test_macos()
+ elif 'linux' in platform_id:
+ return os_test_linux()
+ elif 'windows' in platform_id:
+ return os_test_windows()
+ else:
+ cli.log.warning('Unsupported OS detected: %s', platform_id)
+ return CheckStatus.WARNING
+
+
def os_test_linux():
"""Run the Linux specific tests.
"""
@@ -317,20 +333,8 @@ def doctor(cli):
* [ ] Compile a trivial program with each compiler
"""
cli.log.info('QMK Doctor is checking your environment.')
- status = CheckStatus.OK
-
- # Determine our OS and run platform specific tests
- platform_id = platform.platform().lower()
- if 'darwin' in platform_id or 'macos' in platform_id:
- status = os_test_macos()
- elif 'linux' in platform_id:
- status = os_test_linux()
- elif 'windows' in platform_id:
- status = os_test_windows()
- else:
- cli.log.warning('Unsupported OS detected: %s', platform_id)
- status = CheckStatus.WARNING
+ status = os_tests()
cli.log.info('QMK home: {fg_cyan}%s', QMK_FIRMWARE)