kdenlive0.9-1 is missing xine for DVD preview

Bug #1001646 reported by Sasa Paporovic on 2012-05-19
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
One Hundred Papercuts
Undecided
Unassigned
kdenlive (Ubuntu)
Wishlist
Unassigned

Bug Description

I am on Ubuntu12.10 Quantal 64bit with kdenlive0.9-1.

The configuration assistant on first start is showing at the system check step that "xine" is nedded for "Required to preview your DVD".

When there is no cause against, xine should be added to the kdenlive package dependences and automatically installed together with kdenlive.

ProblemType: Bug
DistroRelease: Ubuntu 12.10
Package: kdenlive 0.9-1
ProcVersionSignature: Ubuntu 3.4.0-2.6-generic 3.4.0-rc7
Uname: Linux 3.4.0-2-generic x86_64
ApportVersion: 2.1-0ubuntu1
Architecture: amd64
Date: Sat May 19 15:23:51 2012
InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Alpha amd64 (20120510)
ProcEnviron:
 TERM=xterm
 PATH=(custom, no username)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: kdenlive
UpgradeStatus: No upgrade log present (probably fresh install)

Sasa Paporovic (melchiaros) wrote :
Changed in kdenlive (Ubuntu):
status: New → Confirmed
Changed in kdenlive (Ubuntu):
status: Confirmed → In Progress
assignee: nobody → Paolo Rotolo (paolorotolo)
Paolo Rotolo (paolorotolo) wrote :

Hi,
Thanks for the bug report.
I've fixed it with this patch.

Regards,
Paolo

Changed in kdenlive (Ubuntu):
status: In Progress → Fix Committed
assignee: Paolo Rotolo (paolorotolo) → nobody
Julian Taylor (jtaylor) wrote :

it appears xine is not required, so it should not go into depends, it should go into recommends or suggests
see: http://www.debian.org/doc/debian-policy/ch-relationships.html#s-binarydeps

Also this applies to debian too.
Please file a bug with an appropriate patch there, ubuntu will then get the change automatically.
https://wiki.ubuntu.com/Debian/Bugs

Changed in kdenlive (Ubuntu):
status: Fix Committed → Triaged

I'm unsubscribing the Sponsors' Team for now. Please re-subscribe when ready.

tags: added: patch
Sasa Paporovic (melchiaros) wrote :

@jtaylor

the problem here was that xine is missing by kdenlive, as it tells itself on very first start of it(see description), so the patch is exactly doing what it should do. It add xine to the dependency to allow kdenlive to do one of its standard functions. If xine is not installed automatically the standard function is broken.

affects: kdenlive → hundredpapercuts

Dear melchiaros and Paolo,

It's true that kdenlive should come with xine installed. However what Julian pointed out is that xine is not a dependency of kdenlive, it should be a recommendation. I.e.: kdenlive does not need xine in order to work properly, it needs it to improve it's functionality. Remember that recommended packages are installed by default.

Julian also said that kdenlive is maintained in Debian (the 'base' of Ubuntu), and this bug should be fixed there. There is no need to increase the delta between Debian and Ubuntu for such a small change.

Changed in kdenlive (Ubuntu):
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers