Bug 520323 - qt-devel must Requires: phonon-devel
Summary: qt-devel must Requires: phonon-devel
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: qt
Version: 11
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-08-30 16:25 UTC by Axel Thimm
Modified: 2009-11-04 12:36 UTC (History)
4 users (show)

Fixed In Version: 4.5.3-7.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-11-04 12:03:55 UTC


Attachments (Terms of Use)

Description Axel Thimm 2009-08-30 16:25:30 UTC
/usr/lib64/pkgconfig/QtWebKit.pc contains:
Libs.private:  -lphonon -lQtDBus -L/lib64 -lQtXml -lQtGui -lQtNetwork -lQtCore

So some builds against qt-devel fail like for example

http://bugzilla.atrpms.net/show_bug.cgi?id=1518

Comment 1 Rex Dieter 2009-08-30 16:51:46 UTC
This will be fixed when (if?) we switch to using qt's phonon. (currently in rawhide, other releases when qt-4.5.3 lands)

Though I'm a little surprised, I thought Libs.private were only used for static linking ?

Comment 2 Axel Thimm 2009-08-30 17:56:55 UTC
(In reply to comment #1)
> Though I'm a little surprised, I thought Libs.private were only used for static
> linking ?  

Looks like the dependency surfaces when building for example mythtv 0.22. The build log in the report doesn't look like a static build.

Maybe a pkgconfig bug exporting Libs.private to the external builds?

Comment 3 Rex Dieter 2009-08-30 18:03:46 UTC
Another thing I'll just do is... since we're purposely not including static libs anyway, strip Libs.private from qt's pkgconfig files so consuming apps won't even try to use them.

Comment 4 Rex Dieter 2009-08-30 18:06:35 UTC
Fwiw,

$ pkg-config QtWebKit --libs
-lQtWebKit -lQtDBus -lQtGui -lQtNetwork -lQtXml -lQtCore

$ pkg-config QtWebKit --libs --static
-L/usr/X11R6/lib -L/lib64 -lQtWebKit -lphonon -lQtDBus -lQtGui -lQtNetwork -lQtXml -lQtCore

Implies your build (mythtv) is purposely requesting static linking here.

Comment 5 Axel Thimm 2009-08-30 19:00:23 UTC
It looks like all mythtv does is

using_qtwebkit {
        QT += webkit
        DEFINES += USING_QTWEBKIT
}

(that's all a case independent grep on all *.pro files yields)

Comment 6 Rex Dieter 2009-08-30 19:45:00 UTC
Ah, likely coming from the qmake-fu in %{_libdir}/QtWebKit.prl

Ugh, the circular dependency is a little more madness than I'm willing to partake of, so my only suggestion is to workaround it by adding
BR: phonon-devel
until 4.5.3 lands.

(abusing closed->deferred as that closely matches the plan here)

Comment 7 Axel Thimm 2009-08-30 19:57:26 UTC
(In reply to comment #6)
> so my only suggestion is to workaround it by adding
> BR: phonon-devel
> until 4.5.3 lands.
> 
> (abusing closed->deferred as that closely matches the plan here)  

But if it's marked closed->anything the workaround will never be applied, or did you commit the workaround to CVS?

Comment 8 Rex Dieter 2009-08-30 22:41:43 UTC
As outlined in comment #1, this is fixed in rawhide already (by using phonon from qt directly, not standalone).  This is also planned to propogate back to previous releases (F-10, F-11) when 4.5.3 happens.

The workaround I mentioned was for apps with problems, like mythtv, to (temporarily) add 
BR: phonon-devel

Comment 9 Axel Thimm 2009-08-31 05:45:37 UTC
But isn't adding the workaround to the qt package better (which carries the bug after all) than modifying N clients and later removing the temporary fix (or forgetting it forever there)? It looks like anything using the webkit is affected.

(Unless 4.5.3 into F10/F11 is something imminent - I have no idea how the schedule looks like - in that case waiting for the bug to go away is OK)

Comment 10 Kevin Kofler 2009-08-31 10:02:06 UTC
The problem is that the qt -> phonon -> qt dependency is circular.

Comment 11 Kevin Kofler 2009-08-31 10:04:22 UTC
The problem is that the qt -> phonon -> qt dependency is circular.

Comment 12 Axel Thimm 2009-08-31 20:09:14 UTC
But the dependencies are run-time dependencies of *-devel packages, or not? Does it really matter? Such a circular dependency just means that the packages need to be installed together or not at all.

The circular dependency is a fact, you obviously need phonon-devel with qt-devel (when using webkit), and vice versa.

Comment 13 Rex Dieter 2009-10-03 15:11:06 UTC
Re-opening, confirmed my suspicions regarding the .prl cause (comment #6)

And, we've backtracked/waffled a bit on our plan to use qt's phonon, so I'll at least fix the .prl file causing this particular issue.

Comment 14 Fedora Update System 2009-10-11 23:49:27 UTC
qt-4.5.3-4.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/qt-4.5.3-4.fc11

Comment 15 Fedora Update System 2009-10-14 01:44:50 UTC
qt-4.5.3-4.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update qt'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-10447

Comment 16 Fedora Update System 2009-10-14 01:49:29 UTC
qt-4.5.3-4.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update qt'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-10462

Comment 17 Fedora Update System 2009-11-04 12:03:25 UTC
qt-4.5.3-7.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 18 Fedora Update System 2009-11-04 12:35:38 UTC
qt-4.5.3-7.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.