Package: release.debian.org Severity: normal User: release.debian....@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.sa...@stud.uni-hannover.de> for the patch. + + new patch: 645025-youtube.diff + + -- Ansgar Burchardt <ans...@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.sa...@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
