Bug 379255 – x11-libs/qt-qt3support[kde,phonon] should not depend on media-libs/phonon[gstreamer]

• Status: RESOLVED DUPLICATE of bug 357531
• Product: Gentoo Linux

[Description Luke-Jr 2011-08-15 12:06:46 UTC]

Once again, the DEPENDs force GNOME-stack gstreamer on KDE instead of something more platform-neutral like VLC...

[Comment 1 Jorge Manuel B. S. Vicetto (RETIRED) 2011-08-16 11:53:06 UTC]

    phonon? (
        !kde? ( || ( ~x11-libs/qt-phonon-${PV}[aqua=,debug=]
            media-libs/phonon[aqua=,gstreamer] ) )
        kde? ( media-libs/phonon[aqua=,gstreamer] ) )"

The gstreamer use dep seems wrong. That should probably be at most media-libs/phonon[aqua=,gstreamer?].

[Comment 2 Tiger 2011-08-26 19:10:06 UTC]

Not fixed since 5 months ? -> https://bugs.gentoo.org/show_bug.cgi?id=357531

[Comment 3 Tiger 2011-08-27 00:27:09 UTC]

Created attachment 284759 [details, diff]
Patch for x11-libs/qt-qt3support-4.7.3

Tiger qt-qt3support # emerge qt-qt3support -pv

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R    ] x11-libs/qt-qt3support-4.7.3-r1  USE="accessibility exceptions kde phonon (-aqua) -debug -gstreamer -pch" 0 kB [1]

Total: 1 package (1 reinstall), Size of downloads: 0 kB
Portage tree and overlays:
 [0] /usr/portage
 [1] /home/tiger/portage

I didn't and I wont try with gstreamer use flag enable, but it should works.

[Comment 4 Otamay 2011-09-09 19:22:36 UTC]

qt-qt3support-4.7.4 is out and has the same problem.

The same patch should work on this new version, the ebuild is (sadly) the same.

Thank you for the patch, by the way!

[Comment 5 Andreas K. Hüttel 2011-09-09 20:16:30 UTC]

Well the reason why it is not getting fixed is likely that noone in the qt team sees this bug... :D 

Reassigning... @qt: what do you think?

[Comment 6 Davide Pesavento (RETIRED) 2011-09-09 21:19:38 UTC]

Has anyone actually tried to use qt-qt3support without phonon-gstreamer?

[Comment 7 Davide Pesavento (RETIRED) 2011-09-09 21:26:56 UTC]

The dependency on media-libs/phonon[gstreamer] was added because of bug #272649, but that might no longer be relevant on more recent Qt versions...

[Comment 8 Tiger 2011-09-09 22:51:44 UTC]

Created attachment 286017 [details]
qt-qt3support ./configure opts

As you can see in configure options :

+  -phonon ............ Build the Phonon module.
+  -phonon-backend..... Build the platform phonon plugin.

Both are enable by default.

That's why I added this code in the ebuild :

if use phonon; then
  myconf="${myconf} -no-phonon-backend"
fi

As gstreamer options is sent to configure script by ebuild if enable. So '-phonon-backend' will force gstreamer support to be compiled.

I think we can maybe do something better like this in ebuild :

if [ use phonon ] && [ use gstreamer ]; then
  myconf="${myconf} -no-phonon-backend"
fi

Now the question comes, is it really useful to keep building Phonon module if no backend enabled ? I have to admit that I didn't check this.
In this case the USE checks in DEPEND will need some modifications.

[Comment 9 Tiger 2011-09-09 23:16:47 UTC]

Can someone that have USE="phonon kde gstreamer" flags enable paste the output of :

  # equery f qt-qt3support

[Comment 10 Tiger 2011-09-09 23:19:51 UTC]

Sorry, forgot the '!' in code :

if [ use phonon ] && [ !use gstreamer ]; then
  myconf="${myconf} -no-phonon-backend"
fi

[Comment 11 Davide Pesavento (RETIRED) 2011-11-18 23:40:53 UTC]

*** This bug has been marked as a duplicate of bug 357531 ***