diff options
| author | Erovia <Erovia@users.noreply.github.com> | 2020-05-21 21:56:43 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-05-21 21:56:43 +0200 |
| commit | 179e5c018fcee9bcb0a7f1907cc96d44ed5cf554 (patch) | |
| tree | eed775035ee27babc2033eafcfb3f932c6808007 /lib/python | |
| parent | 5246acb2e4c188198d503856a493b66174486db8 (diff) | |
| download | qmk_firmware-179e5c018fcee9bcb0a7f1907cc96d44ed5cf554.tar.gz qmk_firmware-179e5c018fcee9bcb0a7f1907cc96d44ed5cf554.zip | |
CLI: Rework submodule checking (#9162)
Diffstat (limited to 'lib/python')
| -rwxr-xr-x | lib/python/qmk/cli/doctor.py | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/lib/python/qmk/cli/doctor.py b/lib/python/qmk/cli/doctor.py index 20f8650e5e..011c3dd3c2 100755 --- a/lib/python/qmk/cli/doctor.py +++ b/lib/python/qmk/cli/doctor.py @@ -24,7 +24,6 @@ ESSENTIAL_BINARIES = { }, 'bin/qmk': {}, } -ESSENTIAL_SUBMODULES = ['lib/chibios', 'lib/lufa'] def _udev_rule(vid, pid=None, *args): @@ -124,17 +123,11 @@ def check_submodules(): for submodule in submodules.status().values(): if submodule['status'] is None: - if submodule['name'] in ESSENTIAL_SUBMODULES: - cli.log.error('Submodule %s has not yet been cloned!', submodule['name']) - ok = False - else: - cli.log.warn('Submodule %s is not available.', submodule['name']) + cli.log.error('Submodule %s has not yet been cloned!', submodule['name']) + ok = False elif not submodule['status']: - if submodule['name'] in ESSENTIAL_SUBMODULES: - cli.log.error('Submodule %s is not up to date!', submodule['name']) - ok = False - else: - cli.log.warn('Submodule %s is not up to date!', submodule['name']) + cli.log.error('Submodule %s is not up to date!', submodule['name']) + ok = False return ok |