DavidoTek / ProtonUp-Qt

Install and manage GE-Proton, Luxtorpeda & more for Steam and Wine-GE & more for Lutris with this graphical user interface.
https://davidotek.github.io/protonup-qt
GNU General Public License v3.0
1.25k stars 40 forks source link

Protonup-qt broke on qt6 #349

Closed robzombie91 closed 8 months ago

robzombie91 commented 8 months ago

Please fill out following when reporting a new bug:

Describe the bug
Protonup-qt will not launch on qt-6 based desktops like the newer plasma 5.90.2 aka plasma 6

To Reproduce
Steps to reproduce the behavior: Install plasma 6 rc install protonup-qt try to launch protonup-qt

Expected behavior
It should launch the protonup-qt window

Screenshots N/A

Desktop (please complete the following information):

Additional context
None

Terminal output

[rob@robpc ~]$ protonup-qt
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/lib/python3.11/site-packages/pupgui2/__main__.py", line 1, in <module>
    from pupgui2.pupgui2 import main
  File "/usr/lib/python3.11/site-packages/pupgui2/pupgui2.py", line 10, in <module>
    from PySide6.QtGui import QIcon, QKeyEvent, QKeySequence, QShortcut
ImportError: /usr/lib/python3.11/site-packages/PySide6/QtGui.abi3.so: undefined symbol: _ZN5QFont11tagToStringEj, version Qt_6
sonic2kk commented 8 months ago

Sounds like an issue with PySide6 / the PySide 6 version rather than ProtonUp-Qt. The latest version is PySide 6.6.3 according to PyPI, and ProtonUp-Qt's requirements.txt requires >= 6.3.0, so I guess the AUR package should be using the latest...

https://github.com/DavidoTek/ProtonUp-Qt/blob/89bdbcf9843210c6e479a2834bde80ed7637c7b4/requirements.txt#L1

Does this happen with the Flatpak or AppImage version out of interest? Also, do any other PySide 6 projects have this error?

It might also be a good idea to mention this on the AUR package repo (protonup-qt), as the AUR package is managed by another person and they may have more insights too. For reference, there are two AUR packages - protonup-qt and protonup-qt-bin, however at time of writing protonup-qt-bin is still on v2.8.2, so I assumed you're using the protonup-qt package :slightly_smiling_face:

sonic2kk commented 8 months ago

Found a similar issue for PySide6 reported a while ago for someone on Kubuntu: https://bugreports.qt.io/browse/PYSIDE-1950. They were getting ImportError: env/lib/python3.8/site-packages/PySide6/QtCore.abi3.so: undefined symbol: _ZNK20QFutureInterfaceBase15isChainCanceledEv, version Qt_6. Not the same error but similar enough that they might be related.

sonic2kk commented 8 months ago

Someone having a similar issue on PyQt5 seemed to resolve it by re-installing the PyQt5 library (https://stackoverflow.com/a/68811630/1952066). Perhaps uninstalling and re-installing PySide6 could help your issue?

Some users had to re-install the python-vdf library to fix an issue with the ProtonUp-Qt AUR package (https://aur.archlinux.org/packages/protonup-qt#comment-932104), so perhaps sometimes Python packages need re-installed?

DavidoTek commented 8 months ago

The issue is normally caused by a mismatch of the ABI versions of the PySide6 version and Qt version.

You can run python3 -c "import PySide6; print(PySide6.__version__)" to get your PySidie6 version. Compare it to the installed Qt version (check the installed packets or run some Qt utility like qtdiag6 to get the version: qtdiag6 | grep Qt)

Does this happen with the Flatpak or AppImage version out of interest

Should not happen here as the ABI versions are matched.

It might also be a good idea to mention this on the AUR package repo (protonup-qt), as the AUR package is managed by another person and they may have more insights too Perhaps uninstalling and re-installing PySide6 could help your issue?

Yes, that is the place to report it. It is maintained separately from the official AppImage and Flatpak releases.

Although, this issue sounds more like a configuration issue with Arch, or a missing update. Try updating your OS.

protonup-qt-bin

This one is using the AppImage version in the background. But as you pointed out, is still back on v2.8.2

robzombie91 commented 8 months ago

The flatpak version doesn't have this issue

DavidoTek commented 8 months ago

I will close this issue as we can't do anything about it from our side.

sonic2kk commented 8 months ago

Indeed, not a ProtonUp-Qt issue, but if you can't fix it (as in, if it's not a system configuration issue) I still encourage you to report it to the package maintainer: https://aur.archlinux.org/packages/protonup-qt :slightly_smiling_face: