Skip to content

Commit

Permalink
Qmk doctor os check to support newer msys2/w10 installations (qmk#8031)
Browse files Browse the repository at this point in the history
* Fixed OS detection on newer MSYS installations

* made OS sting lower case
  • Loading branch information
Duckle29 authored and Erovia committed Jan 28, 2020
1 parent f4ec870 commit 31f4f4e
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions lib/python/qmk/cli/doctor.py
Original file line number Diff line number Diff line change
Expand Up @@ -168,15 +168,15 @@ def doctor(cli):
ok = True

# Determine our OS and run platform specific tests
OS = platform.system() # noqa (N806), uppercase name is ok in this instance
OS = platform.platform().lower() # noqa (N806), uppercase name is ok in this instance

if OS == 'Darwin':
if 'darwin' in OS:
if not os_test_macos():
ok = False
elif OS == 'Linux':
elif 'linux' in OS:
if not os_test_linux():
ok = False
elif OS == 'Windows':
elif 'windows' in OS:
if not os_test_windows():
ok = False
else:
Expand Down

0 comments on commit 31f4f4e

Please sign in to comment.