Bug 279186 - Compile error when build against Qt 4.8
Summary: Compile error when build against Qt 4.8
Status: RESOLVED FIXED
Alias: None
Product: jovie
Classification: Unmaintained
Component: General (show other bugs)
Version: unspecified
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Jeremy Whiting
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-02 16:37 UTC by Radek Novacek
Modified: 2011-08-10 19:16 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.7.1
Sentry Crash Report:

Attachments
Patch that fixes the build failure (574 bytes, patch)
2011-08-02 16:38 UTC, Radek Novacek 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Radek Novacek 2011-08-02 16:37:48 UTC 
Version:           unspecified (using KDE 4.7.0) 
OS:                Linux

Jovie from kdeaccessibility (didn't know which product it is) doesn't build against Qt 4.8. There are some changes in moc in 4.8
which forbid virtual inheritance of QObject. Virtual inheritance of class XmlTransformerProc from KttsFilterProc is not needed and cause build failure with Qt 4.8. 

Reproducible: Always

Steps to Reproduce:
Just build with Qt 4.8


Expected Results:  
It should build
Comment 1 Radek Novacek 2011-08-02 16:38:26 UTC 
Created attachment 62473 [details]
Patch that fixes the build failure

I'm not 100% sure that the patch is correct, but the build succeeds with it.
Comment 2 Rex Dieter 2011-08-02 16:54:51 UTC 
confirm reported issue, see also related upstream report,
https://bugreports.qt.nokia.com//browse/QTBUG-19717

can't find a kdeaccessibility or jovie bz component to assign though...
Comment 3 Jeremy Whiting 2011-08-02 21:53:39 UTC 
That patch is fine.  I did a similar change on the 4.7 branch that just missed the 4.7.0 tarball.  So it's fixed the same way in 4.7.1 and 4.8 (master branch).  I'll also see what I can do about getting kttsd product to have an alias called jovie since the name has changed quite some time ago.