From 01eed8d5d8dc82703b769206c5c08d07a6042b7b Mon Sep 17 00:00:00 2001 From: Nick Brassel Date: Sun, 3 Dec 2023 09:46:37 +1100 Subject: [PATCH] Dedup the list of tested userspace paths. (#22599) --- lib/python/qmk/userspace.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/python/qmk/userspace.py b/lib/python/qmk/userspace.py index 103f11b99ab..7e4cb847c80 100644 --- a/lib/python/qmk/userspace.py +++ b/lib/python/qmk/userspace.py @@ -12,29 +12,29 @@ from qmk.json_encoders import UserspaceJSONEncoder def qmk_userspace_paths(): - test_dirs = [] + test_dirs = {} # If we're already in a directory with a qmk.json and a keyboards or layouts directory, interpret it as userspace if environ.get('ORIG_CWD') is not None: current_dir = Path(environ['ORIG_CWD']) while len(current_dir.parts) > 1: if (current_dir / 'qmk.json').is_file(): - test_dirs.append(current_dir) + test_dirs[current_dir] = True current_dir = current_dir.parent # If we have a QMK_USERSPACE environment variable, use that if environ.get('QMK_USERSPACE') is not None: current_dir = Path(environ['QMK_USERSPACE']) if current_dir.is_dir(): - test_dirs.append(current_dir) + test_dirs[current_dir] = True # If someone has configured a directory, use that if cli.config.user.overlay_dir is not None: current_dir = Path(cli.config.user.overlay_dir) if current_dir.is_dir(): - test_dirs.append(current_dir) + test_dirs[current_dir] = True - return test_dirs + return list(test_dirs.keys()) def qmk_userspace_validate(path):