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

Fix build with Qt 6.6. #199

Merged
merged 1 commit into from Oct 7, 2023
Merged

Conversation

BSDKaffee
Copy link
Contributor

QSqlDataBase::exec() has had a deprecation note in the documentation for 12 years and the function has been finally marked as such in Qt 6.6. With -Werror enabled, the build fails due to the deprecation warning.

QSqlDataBase::exec() has had a deprecation note in the documentation for 12 years and the function has been finally marked as such in Qt 6.6. With -Werror enabled, the build fails due to the deprecation warning.
@textbrowser
Copy link
Owner

:)

This is an automatic-completion error. Thanks.

@textbrowser textbrowser merged commit 95c0d32 into textbrowser:master Oct 7, 2023
@textbrowser textbrowser changed the title Fix build with Qt 6.6 Fix build with Qt 6.6. Oct 7, 2023
vishwin pushed a commit to vishwin/freebsd-ports that referenced this pull request Nov 3, 2023
This port builds with -Werror, and due to QSqlDataBase::exec() being
finally marked as deprecated in Qt 6.6.0 [1], it fails to build due to
deprecation warnings. Add upstreamed patch to fix this issue. [2] Port
builds perfectly fine with previous versions of Qt, including 5.x, as it
took 12 years for the soft deprecation in the API notes [3] to actually
take effect. Upstream notes this was an automatic-completion error. [2]

While here: clean up USE_QT for both the qt5 and qt6 flavors and
remove checks for long unsupported (3+ years) versions of Qt 5.x. [4]

[1] https://doc.qt.io/qt-6/qsqldatabase-obsolete.html
[2] textbrowser/dooble#199
[3] https://doc.qt.io/qt-5/qsqldatabase.html

Approved by:	portmgr (blanket)
With hat:	kde@ [4]
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.

None yet

2 participants