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

Bug#657722: pu: package clive/2.2.13-5+squeeze4



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: pu

Hi,

clive needs another update to handle changes to youtube.com.  The
problem was initially reported (and fixed in unstable) in October[1],
but when I looked into preparing an update to stable the change was
reverted again.

Now the change was activated again and clive/squeeze fails to download
videos again.  I have this backported the change applied to clive in
unstable last year to the version in squeeze (debdiff attached).
Retrieving videos then works again for me.

Please consider also making the update available via stable-updates.

Regards,
Ansgar

[1] <http://bugs.debian.org/645025>
diff -Nru clive-2.2.13/debian/changelog clive-2.2.13/debian/changelog
--- clive-2.2.13/debian/changelog	2011-08-04 20:26:44.000000000 +0200
+++ clive-2.2.13/debian/changelog	2012-01-28 11:42:30.000000000 +0100
@@ -1,3 +1,11 @@
+clive (2.2.13-5+squeeze4) stable; urgency=low
+
+  * Adapt for youtube.com changes. (Closes: #645025)
+    Thanks to Ingo Saitz <Ingo.Saitz@stud.uni-hannover.de> for the patch.
+    + new patch: 645025-youtube.diff
+
+ -- Ansgar Burchardt <ansgar@debian.org>  Sat, 28 Jan 2012 11:42:24 +0100
+
 clive (2.2.13-5+squeeze3) stable; urgency=low
 
   * Adapt for youtube.com changes. (Closes: #636612)
diff -Nru clive-2.2.13/debian/patches/645025-youtube.diff clive-2.2.13/debian/patches/645025-youtube.diff
--- clive-2.2.13/debian/patches/645025-youtube.diff	1970-01-01 01:00:00.000000000 +0100
+++ clive-2.2.13/debian/patches/645025-youtube.diff	2012-01-28 11:41:06.000000000 +0100
@@ -0,0 +1,31 @@
+From: Ingo Saitz <Ingo.Saitz@stud.uni-hannover.de>
+Date: Tue, 11 Oct 2011 22:56:27 +0200
+Subject: Fix problem with some youtube.com videos
+Bug: https://sourceforge.net/apps/trac/clive/ticket/14
+Bug-Debian: http://bugs.debian.org/645025
+
+[ backported to Squeeze with some whitespace changes ]
+
+--- clive.orig/lib/clive/Host/Youtube.pm
++++ clive/lib/clive/Host/Youtube.pm
+@@ -33,7 +33,7 @@
+ 
+     my %re = (
+         id => qr|"video_id": "(.*?)"|,
+-        url_encoded_fmt_stream_map => qr|url_encoded_fmt_stream_map=(.*?)&|,
++        url_encoded_fmt_stream_map => qr|"url_encoded_fmt_stream_map": "(.*?)"|,
+     );
+ 
+     my $tmp;
+@@ -44,9 +44,9 @@
+ 
+         require URI::Escape;
+ 
+-        foreach (split /,/, URI::Escape::uri_unescape ($tmp->{url_encoded_fmt_stream_map})) {
++        foreach (split /,/, $tmp->{url_encoded_fmt_stream_map}) {
+ 	    my %map;
+-	    for my $kv (split /&/) {
++	    for my $kv (split /\\u0026/) {
+ 		my ($key, $value) = split /=/, $kv;
+ 		$map{$key} = URI::Escape::uri_unescape($value);
+ 	    }
diff -Nru clive-2.2.13/debian/patches/series clive-2.2.13/debian/patches/series
--- clive-2.2.13/debian/patches/series	2011-08-04 20:24:52.000000000 +0200
+++ clive-2.2.13/debian/patches/series	2012-01-28 11:33:11.000000000 +0100
@@ -5,3 +5,4 @@
 0001-Youtube-Fix-video-ID-parsing-10.patch
 liveleak-fix-from-2.2.25.patch
 636612-youtube.diff
+645025-youtube.diff

Reply to: