From 12f308748f3dd7ab2a1f3d93f8f207cb51e306a4 Mon Sep 17 00:00:00 2001 From: Ryan Date: Fri, 2 Apr 2021 21:44:27 +1100 Subject: [PATCH] qmk doctor: Display macOS and Windows versions, and add WSL warning for /mnt (#12441) --- lib/python/qmk/cli/doctor.py | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/lib/python/qmk/cli/doctor.py b/lib/python/qmk/cli/doctor.py index 9210737f1e1..45a53c7d44e 100755 --- a/lib/python/qmk/cli/doctor.py +++ b/lib/python/qmk/cli/doctor.py @@ -35,6 +35,11 @@ def os_test_linux(): if 'microsoft' in platform.uname().release.lower(): cli.log.info("Detected {fg_cyan}Linux (WSL){fg_reset}.") + # https://github.com/microsoft/WSL/issues/4197 + if QMK_FIRMWARE.startswith("/mnt"): + cli.log.warning("I/O performance on /mnt may be extremely slow.") + return CheckStatus.WARNING + return CheckStatus.OK else: cli.log.info("Detected {fg_cyan}Linux{fg_reset}.") @@ -46,7 +51,7 @@ def os_test_linux(): def os_test_macos(): """Run the Mac specific tests. """ - cli.log.info("Detected {fg_cyan}macOS{fg_reset}.") + cli.log.info("Detected {fg_cyan}macOS %s{fg_reset}.", platform.mac_ver()[0]) return CheckStatus.OK @@ -54,7 +59,8 @@ def os_test_macos(): def os_test_windows(): """Run the Windows specific tests. """ - cli.log.info("Detected {fg_cyan}Windows{fg_reset}.") + win32_ver = platform.win32_ver() + cli.log.info("Detected {fg_cyan}Windows %s (%s){fg_reset}.", win32_ver[0], win32_ver[1]) return CheckStatus.OK @@ -71,11 +77,10 @@ def doctor(cli): * [ ] Compile a trivial program with each compiler """ cli.log.info('QMK Doctor is checking your environment.') + cli.log.info('QMK home: {fg_cyan}%s', QMK_FIRMWARE) status = os_tests() - cli.log.info('QMK home: {fg_cyan}%s', QMK_FIRMWARE) - # Make sure our QMK home is a Git repo git_ok = check_git_repo()