[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#854619: unblock: svtplay-dl/1.9.1-0.1



Control: tag -1 - moreinfo

On Wed, Feb 08, 2017 at 08:11:25PM +0000, Jonathan Wiltshire wrote:
> On Wed, Feb 08, 2017 at 07:05:40PM +0100, Mattia Rizzolo wrote:
> > Please consider unblocking the package svtplay-dl
> > 
> > Well, the delta is quite small despite being a new upstream release, and
> > it is needed to make the package usable again.
> 
> I disagree, the bug it closes specifically mentions a targetted fix. Your
> upload is a much more invasive new upstream. Does the targetted fix not
> work?

It probably does, but
1) the relevant commit comes after several dozen other commits since the
   version currently in stretch (actually it happened "right before" the
   1.9.1 release); I can't assume that just because the patch applies it
   doesn't depend on some subtle change done before
2) if possible, I always prefer something that an upstream says it's
   "good" rather than mess with patches, especially in small projects
   like this where that's actually feasible
3) if you glances at the upstream commit log you can see there are at
   least 3 more crash fixes, and I deem crash important enough to be
   fixed if possible

Furthermore this is a leaf package, and even if it's true the Debian
freeze doesn't distinguish between leaf and non-leaf or priority of the
package I hope you can see how there is no way this update could
jeopardize the final release.

> We find packages like this are often broken by service API changes during a
> stable lifetime; is svtplay-dl suitable for a stable release?

Another version is available in jessie, and ttbomk everything went well,
so I'd say it is, yes.  Anyway, this is a question for the actual
maintainer :)

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

Attachment: signature.asc
Description: PGP signature


Reply to: