-
Notifications
You must be signed in to change notification settings - Fork 37
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
Qt6-related changes broken build with Qt4 #117
Comments
* Replaced QRegExp with QRegularExpression in tests * Replace most QRegExp occurences with QRegularExpression * Attempt to migrate last QRegExp I'm not sure about the implementation, gotta make some debug tests first to ensure it doesn't break any functionality. * Fix match logic and tests
Qt4 is so old I wonder why it would make sense to fix the bug. What is your reason for Qt4? |
@Thaodan Thank you for responding. The reason being Qt5 is broken for macOS PowerPC (likely on older Intel too). P. S. Just in case, one does not need an old macOS or PowerPC to test Qt4 builds though. It works all through the latest macOS: https://ports.macports.org/port/qt4-mac/details |
For the record, 3.6.0 builds fine. |
The text was updated successfully, but these errors were encountered: