Project

Profile

Help

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...

Bug #870968

deprecated-declarations warnings with Qt-5.14

Added by Marko Lindqvist 9 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Category:
Bootstrap
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Trying to compile S3_0 against development version of crosser-1.10, which has Qt-5.14.2 results in deprecated-declarations errors (warnings turned to errors by -Werror).
This is despite the fact that we have QT_DISABLE_DEPRECATED_BEFORE at Qt-5.2 level only. Qt seems to have no mechanism to allow deprecations above certain version level - that's either a missing feature or a bug, but apparently we need to live with it.

Setting '-Wno-error=deprecated-declarations', or maybe just '-Wno-deprecated-declarations', for --enable-debug builds should work as a quick workaround, and the only one I have figured out so far.

So far this issue has been seen with crosser build only. As such it would affected only S3_0 and later. S2_6 might very well be affected, but I'd rather backport the workaround only when the issue has been confirmed on that branch.

History

#1 Updated by Marko Lindqvist 9 months ago

The simpler solution, -Wno-deprecated-declarations, works. I think this one is fine even for S2_6.

#2 Updated by Marko Lindqvist 9 months ago

FWIW The deprecated declarations warnings are given only about QDesktopWidget::screenGeometry() stuff we resolve (for master) in Feature #871396

#3 Updated by Marko Lindqvist 9 months ago

I think we should push this patch (silencing of the warning) to stable branches, where the deprecation is not going to get fixed, only. Leave it out from master, and instead follow how the situation develops with the future Qt versions.

#4 Updated by Marko Lindqvist 9 months ago

  • Status changed from Resolved to Closed
  • Assignee set to Marko Lindqvist

Pushed to S2_6 & S3_0

Also available in: Atom PDF