Do not auto-detect PySide6 for now

There are still multiple unsolved issues. We leave it in explicitly though, so further testing and integration remains simple.
karlch · Jul 12, 2023 · ee28b6f · ee28b6f
1 parent 3eb336c
commit ee28b6f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/tox.ini b/tox.ini
@@ -31,7 +31,6 @@ ignore_errors = true
 deps =
     -r{toxinidir}/misc/requirements/requirements_pyqt5.txt
     -r{toxinidir}/misc/requirements/requirements_pyqt6.txt
-    -r{toxinidir}/misc/requirements/requirements_pyside6.txt
     -r{toxinidir}/misc/requirements/requirements_piexif.txt
     -r{toxinidir}/misc/requirements/requirements_lint.txt
 commands =

diff --git a/vimiv/qt/__init__.py b/vimiv/qt/__init__.py
@@ -23,6 +23,7 @@ class WRAPPERS(enum.Enum):
     PySide6 = "PySide6"
 
 
+_WRAPPERS_PYQT = [WRAPPERS.PyQt5, WRAPPERS.PyQt6]
 _WRAPPER_NAMES = [wrapper.value for wrapper in WRAPPERS]
 _WRAPPER_NAMES_STR = ", ".join(_WRAPPER_NAMES)
 
@@ -55,8 +56,11 @@ def _select_wrapper():
 
 
 def _autoselect_wrapper():
-    """Select a python Qt wrapper version based on availability and enum order."""
-    for wrapper in WRAPPERS:
+    """Select a python Qt wrapper version based on availability and enum order.
+
+    We do not auto-detect PySide6 yet, as there are still multiple issues.
+    """
+    for wrapper in _WRAPPERS_PYQT:
         try:
             importlib.import_module(wrapper.value)
             return wrapper