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

qt: switch to PySide6 #691

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

achow101
Copy link
Member

@achow101 achow101 commented Jul 25, 2023

PySide2 uses Qt5 which is no longer maintained. PySide6 uses Qt6 and is the current version of Qt, so we should switch to that. This also lets us build the GUI for Apple Silicon macs.

TODOs:

  • Binary builds (debian oldoldstable doesn't have qt6)

@achow101 achow101 marked this pull request as draft July 25, 2023 16:35
@achow101 achow101 changed the title qt: Fixes and switch to PySide6 qt: switch to PySide6 Jul 27, 2023
@brunoerg
Copy link
Contributor

Tested it on Mac M1 and worked fine.

@brunoerg
Copy link
Contributor

You missed to change line 242:

self.ui.path_lineedit.setValidator(QRegularExpressionValidator(QRegExp(r"m(/[0-9]+['Hh]?)+/\*"), None))

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

Successfully merging this pull request may close these issues.

2 participants