[jaunty] Presets should be updated because the ffmpeg version in Ubuntu is newer than in Debian

Bug #304249 reported by Paul Gevers
4
Affects Status Importance Assigned to Milestone
winff (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The presets.xml file that comes with winff is compatible with the version of ffmpeg in debian. However the version in jaunty is a lot newer. The presets file is just a text file with ffmpeg command lines which the users can select using the GUI that WinFF provides to use ffmpeg. ffmpeg has seen some changes in the recent past, justifying an updated presets file. Also because the libavcodec-unstripped-52 supports a lot more than debians libavcodec.

Additionally, because Ubuntu provides the unstripped libraries in multiverse, a recommands should be added to obtain the best user experience.

I attached a debdiff to accomplish the update, with the following changelog:

winff (0.43-2ubuntu1) jaunty; urgency=low

  * Changed debian/preset.xml file to be in line with ffmpeg in jaunty.
  * Removed debian/presets_new.xml because already included in the
    debian/presets.xml
  * Updated debian/README.debian
  * Removed absolute path and extension from debian/desktop
  * Added recommends on (k|x|)ubuntu-restricted-extras to control file to
    support presets.

Tags: patch update
Revision history for this message
Paul Gevers (paul-climbing) wrote :
Revision history for this message
Paul Gevers (paul-climbing) wrote :

I set the bug to confirmed as I am pretty sure of this.

Changed in winff:
status: New → Confirmed
description: updated
Revision history for this message
James Westby (james-w) wrote :

Hi,

I'm not sure we should be recommending the restricted-extras packages.
I'm not sure if there is a concrete policy on this, a suggests may be ok.
What would happen if I sponsored the rest of the changes without that one?

Thanks,

James

Revision history for this message
Paul Gevers (paul-climbing) wrote : Re: [Bug 304249] Re: [jaunty] Presets should be updated because the ffmpeg version in Ubuntu is newer than in Debian

Sorry for taking so long to respond, I had a long holiday.

> I'm not sure we should be recommending the restricted-extras packages.
> I'm not sure if there is a concrete policy on this, a suggests may be ok.
> What would happen if I sponsored the rest of the changes without that one?

Basically that would be fine of course. But it would be really nice if
people that have the multiverse repositories enabled would pull in the
unstripped versions of ffmpeg, because without it most presets are
useless (that is why Debian has such a small amount of presets). On the
winff forums [1] most questions are when people are using a restricted
version of ffmpeg and are not able to convert the files to the type they
request. So indeed, at the very least have the suggests (I assume that
using Suggests would follow Debian-policy on the Recommends/Suggests).

Unfortunately the policies and license issues with ffmpeg are not very
understandable for the novice users. Would it be appropriate to mention
this in the description of the winff package?

Paul

[1] http://winff.org/forums

Revision history for this message
James Westby (james-w) wrote :

On Sat, 2009-01-03 at 16:48 +0000, Paul Gevers wrote:
> Sorry for taking so long to respond, I had a long holiday.
>
> > I'm not sure we should be recommending the restricted-extras packages.
> > I'm not sure if there is a concrete policy on this, a suggests may be ok.
> > What would happen if I sponsored the rest of the changes without that one?
>
> Basically that would be fine of course. But it would be really nice if
> people that have the multiverse repositories enabled would pull in the
> unstripped versions of ffmpeg, because without it most presets are
> useless (that is why Debian has such a small amount of presets). On the
> winff forums [1] most questions are when people are using a restricted
> version of ffmpeg and are not able to convert the files to the type they
> request. So indeed, at the very least have the suggests (I assume that
> using Suggests would follow Debian-policy on the Recommends/Suggests).

I believe that would be fine.

> Unfortunately the policies and license issues with ffmpeg are not very
> understandable for the novice users. Would it be appropriate to mention
> this in the description of the winff package?

This is a good idea. Would you like to propose an updated diff with
these changes?

Thanks,

James

Revision history for this message
Paul Gevers (paul-climbing) wrote :

> This is a good idea. Would you like to propose an updated diff with
> these changes?

Will do that tomorrow.

Paul

Revision history for this message
Paul Gevers (paul-climbing) wrote :

I have created a new debdiff with the discussed changes: suggests instead of recommends and extra text in the description.

I added a quick fix to Debian bug #510156 by adding xterm as a dependency.

I hope you can agree with the changes now.

Changelog:
winff (0.43-2ubuntu1) jaunty; urgency=low

  * Changed debian/preset.xml file to be in line with ffmpeg in jaunty.
  * Removed debian/presets_new.xml because already included in the debian/presets.xml
  * Updated debian/README.debian
  * Removed absolute path and extension from debian/desktop
  * Added suggest on (k|x|)ubuntu-restricted-extras to control file to support presets.
  * Added text to describtion to explain that the a user needs to install libavcodec-unstripped-XX to use all presets.
  * Added dependency on xterm (See Debian bug 510156)

 -- Paul Gevers <email address hidden> Mon, 05 Jan 2009 20:06:57 +0100

Revision history for this message
Paul Gevers (paul-climbing) wrote : debian updated

I just updated the Debian package, so this bug fix can wait until winff
is synced from Debian.

I will create a debdiff for the new version of winff.

Paul

Revision history for this message
Paul Gevers (paul-climbing) wrote :

The debdiff became a lot smaller with version 0.45.1-1. It is attached.

Changelog:
  * Changed rules file to install presets in line with ffmpeg in jaunty.
  * Updated debian/README.debian
  * Added suggest on (k|x|)ubuntu-restricted-extras to control file to support presets.
  * Added text to description to explain that the a user needs to install libavcodec-unstripped-XX to use all presets.

Emmet Hikory (persia)
Changed in winff:
status: Confirmed → Triaged
Revision history for this message
Paul Gevers (paul-climbing) wrote :

James (or any other possible sponsor),

Would you mind commenting on my last debdiff? If possible of course just implementing ;)

Paul

James Westby (james-w)
Changed in winff:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.