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

hooks: qt: prevent potential error due to partially-installed PyQt6 #6141

Merged
merged 1 commit into from
Aug 22, 2021

Conversation

rokm
Copy link
Member

@rokm rokm commented Aug 19, 2021

Prevent potential errors on importing PyInstaller.utils.hooks.qt module, caused by a partial PyQt6 installation (e.g., only
PyQt6-Qt6 and PyQt6-sip remaining installed after user uninstalls PyQt6 package). As the error is raised on import of the qt hookutils module, it then affects all hooks for all Qt-based packages (not just PyQt6).

Prevent potential errors on importing PyInstaller.utils.hooks.qt
module, caused by a partial PyQt6 installation (e.g., only
PyQt6-Qt6 and PyQt6-sip remaining installed after user uninstalls
PyQt6 package).
@rokm
Copy link
Member Author

rokm commented Aug 19, 2021

This change is similar to the one in #6114 for PyQt5, but in this case the underlying reason is a bit different (although now that I think about it, one could end up with similar partial installation of PyQt5 as well).

@rokm rokm merged commit 6cb9926 into pyinstaller:develop Aug 22, 2021
@rokm rokm deleted the partial_pyqt6_safeguard branch August 22, 2021 09:38
rokm added a commit that referenced this pull request Aug 22, 2021
…6141)

Prevent potential errors on importing PyInstaller.utils.hooks.qt
module, caused by a partial PyQt6 installation (e.g., only
PyQt6-Qt6 and PyQt6-sip remaining installed after user uninstalls
PyQt6 package).
@rokm rokm added the backported Backported to v4 branch label Sep 26, 2021
@bwoodsend bwoodsend mentioned this pull request Feb 3, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backported Backported to v4 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants