Bug 1098191

Summary: doxywizard will not build with qt 5.11
Product: [openSUSE] openSUSE Tumbleweed Reporter: Petr Gajdos <pgajdos>
Component: KDE ApplicationsAssignee: Petr Gajdos <pgajdos>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Normal    
Priority: P5 - None    
Version: Current   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: Patch to fix build

Description Petr Gajdos 2018-06-19 10:01:10 UTC
Following patch does fixes the build for Tumbleweed:
https://build.opensuse.org/package/view_file/devel:tools/doxygen/doxywizard-qt-5.11.patch?expand=1
Comment 1 Petr Gajdos 2018-06-19 10:13:42 UTC
However, I would like to ask you, Luca and Michal:

Is this patch correct for qt 5.11?

If yes, can it be base for upstream patch? I mean: will this work for older Qt5 than 5.11? It is clear qt_use_modules macro definition will be redundant in that case, but it will still be needed for Qt4 I guess?

I regret I do not know cmake more, could you help me please?
Comment 2 Christophe Giboudeaux 2018-06-19 10:51:20 UTC
The patch is fine for our package and won't break build with older Qt versions.
It would break the Qt4 build, however.
Comment 3 Christophe Giboudeaux 2018-06-19 10:52:54 UTC
Created attachment 774473 [details]
Patch to fix build

Here's a patch you can use for this package and submit upstream.
Comment 4 Petr Gajdos 2018-06-19 13:18:24 UTC
Thanks, package submitted to Tumbleweed. Corresponding pull request:
https://github.com/doxygen/doxygen/pull/772