diff options
| author | Cristián Maureira-Fredes <Cristian.Maureira-Fredes@qt.io> | 2024-12-03 10:03:24 +0100 |
|---|---|---|
| committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2024-12-04 09:34:48 +0000 |
| commit | eb35f1ea8dde0836a3501cd80dc45f9171977197 (patch) | |
| tree | 797865e8116e3987cdeac544748b9f6cee1c7803 /sources/pyside-tools/deploy_lib | |
| parent | 6e887815406753356e0d88e4ee65bfe4a0b31faf (diff) | |
deploy: use tuple instead of list for lru_cache function
One cannot use list in the arguments of a function using lru_cache
Fixes: PYSIDE-2945
Change-Id: I2305dab868a634bfb8a3f240d403c07281f7edb8
Reviewed-by: Shyamnath Premnadh <Shyamnath.Premnadh@qt.io>
(cherry picked from commit 0c0aa19cdddc8c71ad68993061559b66cd1adb46)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'sources/pyside-tools/deploy_lib')
| -rw-r--r-- | sources/pyside-tools/deploy_lib/dependency_util.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sources/pyside-tools/deploy_lib/dependency_util.py b/sources/pyside-tools/deploy_lib/dependency_util.py index e99d654c7..e614576b9 100644 --- a/sources/pyside-tools/deploy_lib/dependency_util.py +++ b/sources/pyside-tools/deploy_lib/dependency_util.py @@ -18,7 +18,7 @@ from . import IMPORT_WARNING_PYSIDE, DEFAULT_IGNORE_DIRS, run_command @lru_cache(maxsize=None) -def get_py_files(project_dir: Path, extra_ignore_dirs: list[Path] = None, project_data=None): +def get_py_files(project_dir: Path, extra_ignore_dirs: tuple[Path] = None, project_data=None): """Finds and returns all the Python files in the project """ py_candidates = [] @@ -116,6 +116,8 @@ def find_permission_categories(project_dir: Path, extra_ignore_dirs: list[Path] return set(perm_categories) + if extra_ignore_dirs is not None: + extra_ignore_dirs = tuple(extra_ignore_dirs) py_candidates = get_py_files(project_dir, extra_ignore_dirs, project_data) for py_candidate in py_candidates: all_perm_categories = all_perm_categories.union(pyside_permission_imports(py_candidate)) @@ -172,6 +174,8 @@ def find_pyside_modules(project_dir: Path, extra_ignore_dirs: list[Path] = None, return set(modules) + if extra_ignore_dirs is not None: + extra_ignore_dirs = tuple(extra_ignore_dirs) py_candidates = get_py_files(project_dir, extra_ignore_dirs, project_data) for py_candidate in py_candidates: all_modules = all_modules.union(pyside_module_imports(py_candidate)) |