FS#27670 - [qtwebkit] Package depends on phonon but doesn't need it

Attached to Project: Arch Linux
Opened by Arcaik (arcaik) - Thursday, 22 December 2011, 17:20 GMT
Last edited by Andrea Scarpino (BaSh) - Thursday, 22 December 2011, 21:45 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Andrea Scarpino (BaSh)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Hi,

The QtWebkit package depends on GStreamer *and* on Phonon.

I think GStreamer dependency should be removed because Phonon provides its own backends (such as phonon-xine, phonon-gstreamer, phonon-mplayer, etc.).

On the IRC channel (#archlinux), someone told me that « GStreamer was the only sane way to read videos ». If that's the reason of that dependency, I think it would be great to respect users' choice. Also, phonon-gstreamer is the default choice for phonon-backend so users who don’t know, would use it.

(Sorry for bad english, I'm french)
This task depends upon

Closed by  Andrea Scarpino (BaSh)
Thursday, 22 December 2011, 21:45 GMT
Reason for closing:  Fixed
Additional comments about closing:  qtwebkit 2.2.1-1
Comment by Ionut Biru (wonder) - Thursday, 22 December 2011, 17:49 GMT
gstreamer is a blessed dependency in kde
Comment by Jan de Groot (JGC) - Thursday, 22 December 2011, 18:57 GMT
GStreamer is a requirement for <video> tag support. The library is linked straight into libqtwebkit.so, so there's no way to make this optional without breaking video support. If you don't want GStreamer on your system, recompile qtwebkit without video support.
Comment by Jan de Groot (JGC) - Thursday, 22 December 2011, 19:03 GMT
About Phonon: you have a point here:
ChangeLog: [Qt] Remove Phonon MediaPlayer from the tree.

Besides the changelog and some testsuite stuff, I don't see any reference to phonon at all. IMHO Phonon should get dropped from depends instead.
Comment by Andrea Scarpino (BaSh) - Thursday, 22 December 2011, 21:35 GMT
I'm trying to rebuild qtwebkit without phonon. I don't use to put dependencies if they aren't needed during the build or if they aren't specified in the README/INSTALL file, but ATM I don't remember why phonon is there.
Comment by Jan de Groot (JGC) - Thursday, 22 December 2011, 21:43 GMT
Probably left behind from some old PKGBUILD. Phonon used to be a dependency for this package, but it was deactivated and later removed as far as I can understand from the upstream ChangeLog.
Comment by Andrea Scarpino (BaSh) - Thursday, 22 December 2011, 21:43 GMT
Seems so. I just built 2.2.1 without phonon.

Loading...