Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash when importing QtCore with PySide 6.8 #494

Open
maflAT opened this issue Oct 11, 2024 · 5 comments
Open

Crash when importing QtCore with PySide 6.8 #494

maflAT opened this issue Oct 11, 2024 · 5 comments

Comments

@maflAT
Copy link

maflAT commented Oct 11, 2024

With PySide 6.8.0 installed, qtpy will crash without raising an Exception when importing QtCore from qtpy.
The crash happens in "./qtpy/QtCore.py", line 128: from PySide6.QtCore import *.

@ccordoba12
Copy link
Member

ccordoba12 commented Oct 11, 2024

Hey @maflAT, thanks for reporting. This seems similar to issue #480, in the sense that PySide 6.8 seems to have broken star imports again (i.e. imports of the form from PySide6.QtCore import *).

Since Qtpy is simply a wrapper around PyQt and PySide, there's nothing we can do about it. Instead, please report this problem in the Qt bug tracker and kindly ask them to run our test suite before releasing a new minor version to avoid this problem in the future.

@hmaarrfk
Copy link
Contributor

xref: https://bugreports.qt.io/browse/PYSIDE-2888

@owillebo
Copy link

Note that the crash is a CPython crash, i.e. an access violation (exit code 0xC0000005) on windows, i.e.;

import qtpy.QtCore

crashes with 0xC0000005 and;

import PySide6.QtCore

exits normally with exit code 0.

See attached test.bat file.

This bat file outputs;

Python 3.11.9
packaging==24.1
PySide6==6.8.0
PySide6_Addons==6.8.0
PySide6_Essentials==6.8.0
QtPy==2.4.1
shiboken6==6.8.0
QtPy exit code: -1073741819
PySide6 exit code: 0

test.zip

@hmaarrfk
Copy link
Contributor

My guess is that the 6.8.0.1 from pypi resolved things they released it today.

On conda-forge we also released a backport though we called it 6.8.0 build 1
conda-forge/pyside2-feedstock#249

Upstream seems to have added a test for star imports too.

@hmaarrfk
Copy link
Contributor

In the fact the bug report I originally opened refers to 6.8.0.1 as having the fix:
https://bugreports.qt.io/browse/PYSIDE-2888

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants