Bug#725142: pu: package totem-plugin-arte/3.2.1-1~wheezy1

2015-08-29 Thread Adam D. Barratt 
Control: tags -1 + confirmed

On Thu, 2014-10-09 at 21:15 +0200, Nicolas Delvaux wrote:
 Hi,
 
 A year later, the package is still broken in Stable.
 
 Here is, yet again, a new debdiff.
 Compared to the previous one, I edited a regex in order to be
 compatible with a change that recently happened on the Arte Website.

Apologies for the delays. If this something you're still interested in
fixing in wheezy, then please feel free to upload (with wheezy in the
changelog rather than stable).

Regards,

Adam

Bug#725142: pu: package totem-plugin-arte/3.2.1-1~wheezy1

2014-10-09 Thread Nicolas Delvaux 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

A year later, the package is still broken in Stable.

Here is, yet again, a new debdiff.
Compared to the previous one, I edited a regex in order to be
compatible with a change that recently happened on the Arte Website.


Can someone finally upload this?

Thanks.
Nicolas
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQIcBAEBAgAGBQJUNt63AAoJEF5mTbNeG1+eBLwQAImsNXN3WJgsYaydEPyyICew
GZS/2juauQ2P+aWQiWlyShTN8Qhn6EiD3mN0/K9k8x69ao+0O004m8lEhTGXrs2u
4L3VKRu3kzjtZ39Ign0oU19SkC4xB8JSMX55LBxyyzDoADSt3kgkYFoRbAa+h6v2
6/pLEXVZuTVELOLlYgx0K0O5L3DzgWw5mo5QpZNtPRDf59XSq1k6c92EObadNFMZ
ua8NidLdDLfNzA73UH7Rzc/oG+6eeUA3mz4mt5EVqlEZPFhHijpm4HZX4vTx3fgG
cI8AdtRy5GSXcwj0YSF4zJF2fYjIkT7tuKQdphKFYB7FDPZlGTfyc7p8wI90CwdW
9pRoUlDQxTSOoBa2u9CsRi4Sb3qkDXp1i9b3nhKJ7qc38nkhh7Z7HjcryidvjF5v
hko3GecxuhalhMKqtsMiuL+kpGqCDRPs8aaxCwfhS0v5SRrnXw7MJE+X63RN266K
sHMzNRNXO3ufyHbB8cwa6WXISVXujvbz+3QRwy+5GWl6CnMJWCdUy20MVawC8FMk
9WRv6+6hQEdQK/fsCvuOOf5FHF/1BWf8Y4dr0irnCqOayPBGvJFj93LZzMMUexoj
FSazFfwre5KHKdWAr2eT0daAmYzfPpaJzHvg+WP+S7syFP7YPx8+/BvmEpwupn51
ZVbvZDdsGKhKJ7mmgsJy
=oQ6g
-END PGP SIGNATURE-
diff -Nru totem-plugin-arte-3.1.2/debian/changelog totem-plugin-arte-3.1.2/debian/changelog
--- totem-plugin-arte-3.1.2/debian/changelog	2012-09-15 14:15:20.0 +0200
+++ totem-plugin-arte-3.1.2/debian/changelog	2014-10-09 16:22:42.0 +0200
@@ -1,3 +1,10 @@
+totem-plugin-arte (3.1.2-1+deb7u1) stable; urgency=low
+
+  * Update the video URL extractor
+  * Add a JSON parsing method and update the XML parser links
+
+ -- Nicolas Delvaux cont...@nicolas-delvaux.org  Tue, 09 Oct 2014 14:22:42 +
+
 totem-plugin-arte (3.1.2-1) unstable; urgency=medium
 
   * New upstream bug-fix only release
diff -Nru totem-plugin-arte-3.1.2/debian/control totem-plugin-arte-3.1.2/debian/control
--- totem-plugin-arte-3.1.2/debian/control	2012-05-06 13:41:04.0 +0200
+++ totem-plugin-arte-3.1.2/debian/control	2014-10-09 16:22:42.0 +0200
@@ -8,6 +8,7 @@
libgtk-3-dev,
libsoup2.4-dev,
libpeas-dev (= 1.2.0),
+   libjson-glib-dev,
gettext
 Standards-Version: 3.9.3
 Homepage: http://gitorious.org/totem-plugin-arte
diff -Nru totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch
--- totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch	1970-01-01 01:00:00.0 +0100
+++ totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch	2014-10-09 17:51:34.0 +0200
@@ -0,0 +1,207 @@
+Description: Add a JSON parsing method and update the XML parser links
+ This change is backported from the 3.2.1 upstream version.
+ This patch will hopefully harden the plugin against more Arte website updates.
+Author: Nicolas Delvaux cont...@nicolas-delvaux.org
+
+---
+
+Origin: vendor
+Forwarded: not-needed
+Last-Update: 2013-12-19
+
+Index: totem-plugin-arte-3.1.2/arteparser.vala
+===
+--- totem-plugin-arte-3.1.2.orig/arteparser.vala	2014-10-09 17:51:13.806886723 +0200
 totem-plugin-arte-3.1.2/arteparser.vala	2014-10-09 17:51:13.802886723 +0200
+@@ -28,6 +28,7 @@
+ 
+ using GLib;
+ using Soup;
++using Json;
+ 
+ public abstract class ArteParser : GLib.Object
+ {
+@@ -55,7 +56,7 @@
+ return has_data;
+ }
+ 
+-public unowned GLib.SListVideo parse (Language lang) throws MarkupError, IOError
++public virtual unowned GLib.SListVideo parse (Language lang) throws MarkupError, IOError
+ {
+ videos = new GLib.SListVideo ();
+ 
+@@ -105,36 +106,127 @@
+ }
+ }
+ 
++public class ArteJSONParser : ArteParser
++{
++private string json_url_fr = http://www.arte.tv/guide/fr/plus7.json;;
++private string json_url_de = http://www.arte.tv/guide/de/plus7.json;;
++
++public ArteJSONParser ()
++{
++reset ();
++}
++
++public override void reset ()
++{
++has_data = true;
++}
++
++public override uint get_error_threshold ()
++{
++return 1; // no errors are tolerated
++}
++
++public override unowned GLib.SListVideo parse (Language lang) throws MarkupError, IOError
++{
++videos = new GLib.SListVideo ();
++
++Soup.Message msg;
++if (lang == Language.GERMAN) {
++msg = new Soup.Message (GET, json_url_de);
++} else {
++msg = new Soup.Message (GET, json_url_fr);
++}
++
++Soup.SessionAsync session = create_session ();
++
++session.send_message (msg);
++
++if (msg.status_code != Soup.KnownStatusCode.OK) {
++throw new IOError.HOST_NOT_FOUND (videos.arte.tv could not be accessed.);
++}
++
++var parser = new Json.Parser ();
++
++try {
++parser.load_from_data ((string) msg.response_body.flatten ().data, -1);
++} catch (GLib.Error e) {
++throw new

Bug#725142: pu: package totem-plugin-arte/3.2.1-1~wheezy1

2014-02-17 Thread Nicolas Delvaux 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,


Is there anything I can do to help the validation of the fixed package?
As a reminder, the totem-plugin-arte package is broken in Stable since
mid-August...

Thanks,
Nicolas
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.14 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJTAphNAAoJEF5mTbNeG1+eOqMQAJIWdAKthLH2fp4dTYDDXvOg
hYi4hLbjYk3IzMIyjTy5HsBu5XX9XYmOwe8FnHlIvjWdWFNr2kIszxKn7CuEwwE+
AVn3D/ybEAHLbswPg0eESq2cAaeGpgzjcRejHJYkA8jPfbgSDh8M1XAaNIglhqAT
LKlDPDo21wDYUWc2UdqfQOnRNKeamy+HhW0jPAjQVz+F2hdyWtvUVPkbBQkJImY/
+qEaeCf1XAFrpe7y+WRLw3BkxZxsR/QXKtRKOeWnQ98iKNZjV+nwfngYYk3USL/M
ry1ek1tEEvQIRYONgVqi61j1xH87P82uKOsDkXf7g5NhrCOf6ctN3JtTFl3DE0AK
2tA1E+jeVYock4RadNomXskzflkXepx1NR9hm5PXA2q88UOZDBWTx+Wm+D32ztNY
0INXquKLcmMla47aggG42JQkHkh6+tP2Wvs1X/BFkwKwUfiIoSmqCsR9Cm6FvTbP
NnS611uUAI70iv9JK+0m1QW88pgYz3hX5Ioh7IMBLEKa+oIA+tLAonBiQ4yxl1uI
d9G58iPp2Qcu1BHGe+2KZc8Rz0m/tYwpU36kWNjWcRSW1QklJFuMvHKI3S2sLvGa
LReOQR/PQvjOZarq3O6zT1kM5L708MN2VF4JQaG4rDp088/ziooMr/R7E1XdL420
XkPgRNfJyq36vIeB/x3e
=zeyR
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#725142: pu: package totem-plugin-arte/3.2.1-1~wheezy1

2014-01-22 Thread intrigeri 
Control: tag -1 - moreinfo

Nicolas Delvaux wrote (19 Dec 2013 22:46:25 GMT) :
 Here is a new version proposal, this time taking the traditional path
 of patching the current stable package.

 The debdiff is shorter/easier to review than the one based on the new
 upstream version, but I would still prefer the 3.2.1-1~deb7u1 version
 mainly because it would be easier to maintain for me.

 However, this fix is pending for a much too long time. Now I just want
 the package to be usable again :-)

 All packages are available on mentors:
 http://mentors.debian.net/package/totem-plugin-arte

AFAICT Nicolas has addressed all concerned raised by the release team,
hence I'm dropping the moreinfo tag.

Cheers,
--
  intrigeri
  | GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
  | OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#725142: pu: package totem-plugin-arte/3.2.1-1~wheezy1

2013-12-19 Thread Nicolas Delvaux 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

Here is a new version proposal, this time taking the traditional path
of patching the current stable package.

The debdiff is shorter/easier to review than the one based on the new
upstream version, but I would still prefer the 3.2.1-1~deb7u1 version
mainly because it would be easier to maintain for me.

However, this fix is pending for a much too long time. Now I just want
the package to be usable again :-)

All packages are available on mentors:
http://mentors.debian.net/package/totem-plugin-arte

Thank you for your help,
Nicolas
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.14 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJSs3dBAAoJEF5mTbNeG1+eLxMP/3BKrQLBDmxTNNDl4/VpiUVF
zh1Sa+oyw0zxSMTUrQsxxe6HHmoP1e/IYwe6RkptZyqTb9EfqDnyQPYlNhlAJDZU
ucMtP8/+1afAg6v+HXrvWrm0s2m3irU78kRkrehGPXehXEnOwqE/euiaAmc0reaw
ltjOIqQHAc59XmYhrcZpumDRFOGFT9hlHpOno2F4SeQeRl7cF7JydAOLTWIfUcGZ
s38ojZkYu7aPEQCq4vTFAaHgyOv4zzOzZgnTEzMp25D6vfO3lp0ag51VJQQ/9VhK
7Ro6u5sSHAV08e9mkO641mX/B3D97hLWXAhJMjkDkj6JALj8E3eInM/GY3QGqPvk
wkGSziwe+0Pf+tNhfzxdFfMfrdRcdO+H/PsGcn2RDdzFYy4Fp9gvGfMxipQ/PIc/
3i93ImP//dQw+InX1EIAXCtgLB6O66mHw6wtGD+Wv6z1gMZafeL7kqG55/fNa4HR
LKt3N88x/zbN37I9counnBdrlqviHA+j2H3GmeR6ffAFwKK1DDU0E+v0QUd/qaVY
jvXGLDcFPjTwhC+S2duUt2GBkPG2LFz4jtEAUhrbBPE1ZGDZh0okbyq6T82duX2O
z09PMUlzIPnopcOiACPpRPu0mGTNT2Hd8LHho67l6ulD22zBqBD5EH6HGuTPHaV7
mY57W4/cdwlwW03TRNQI
=XcNN
-END PGP SIGNATURE-
diff -Nru totem-plugin-arte-3.1.2/debian/changelog totem-plugin-arte-3.1.2/debian/changelog
--- totem-plugin-arte-3.1.2/debian/changelog	2012-09-15 14:15:20.0 +0200
+++ totem-plugin-arte-3.1.2/debian/changelog	2013-12-19 22:49:18.0 +0100
@@ -1,3 +1,10 @@
+totem-plugin-arte (3.1.2-1+deb7u1) stable; urgency=low
+
+  * Update the video URL extractor
+  * Add a JSON parsing method and update the XML parser links
+
+ -- root cont...@nicolas-delvaux.org  Tue, 17 Dec 2013 22:48:14 +
+
 totem-plugin-arte (3.1.2-1) unstable; urgency=medium
 
   * New upstream bug-fix only release
diff -Nru totem-plugin-arte-3.1.2/debian/control totem-plugin-arte-3.1.2/debian/control
--- totem-plugin-arte-3.1.2/debian/control	2012-05-06 13:41:04.0 +0200
+++ totem-plugin-arte-3.1.2/debian/control	2013-12-17 23:53:27.0 +0100
@@ -8,6 +8,7 @@
libgtk-3-dev,
libsoup2.4-dev,
libpeas-dev (= 1.2.0),
+   libjson-glib-dev,
gettext
 Standards-Version: 3.9.3
 Homepage: http://gitorious.org/totem-plugin-arte
diff -Nru totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch
--- totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch	1970-01-01 01:00:00.0 +0100
+++ totem-plugin-arte-3.1.2/debian/patches/parsers_update.patch	2013-12-19 23:07:33.0 +0100
@@ -0,0 +1,180 @@
+Description: Add a JSON parsing method and update the XML parser links
+ This change is backported from the 3.2.1 upstream version.
+ This patch will hopefully harden the plugin against more Arte website updates.
+Author: Nicolas Delvaux cont...@nicolas-delvaux.org
+
+---
+
+Origin: vendor
+Forwarded: not-needed
+Last-Update: 2013-12-19
+
+--- totem-plugin-arte-3.1.2.orig/arteparser.vala
 totem-plugin-arte-3.1.2/arteparser.vala
+@@ -28,6 +28,7 @@
+ 
+ using GLib;
+ using Soup;
++using Json;
+ 
+ public abstract class ArteParser : GLib.Object
+ {
+@@ -55,7 +56,7 @@ public abstract class ArteParser : GLib.
+ return has_data;
+ }
+ 
+-public unowned GLib.SListVideo parse (Language lang) throws MarkupError, IOError
++public virtual unowned GLib.SListVideo parse (Language lang) throws MarkupError, IOError
+ {
+ videos = new GLib.SListVideo ();
+ 
+@@ -105,36 +106,127 @@ public abstract class ArteParser : GLib.
+ }
+ }
+ 
++public class ArteJSONParser : ArteParser
++{
++private string json_url_fr = http://www.arte.tv/guide/fr/plus7.json;;
++private string json_url_de = http://www.arte.tv/guide/de/plus7.json;;
++
++public ArteJSONParser ()
++{
++reset ();
++}
++
++public override void reset ()
++{
++has_data = true;
++}
++
++public override uint get_error_threshold ()
++{
++return 1; // no errors are tolerated
++}
++
++public override unowned GLib.SListVideo parse (Language lang) throws MarkupError, IOError
++{
++videos = new GLib.SListVideo ();
++
++Soup.Message msg;
++if (lang == Language.GERMAN) {
++msg = new Soup.Message (GET, json_url_de);
++} else {
++msg = new Soup.Message (GET, json_url_fr);
++}
++
++Soup.SessionAsync session = create_session ();
++
++session.send_message (msg);
++
++if (msg.status_code != Soup.KnownStatusCode.OK) {
++throw new IOError.HOST_NOT_FOUND (videos.arte.tv could not be accessed.);
++

Bug#725142: pu: package totem-plugin-arte/3.2.1-1~wheezy1

2013-10-01 Thread Nicolas Delvaux 
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: pu


The current totem-plugin-arte package in Wheezy is unusable because of a 
changes on Arte's website.

My package is on mentors.debian.net:

http://mentors.debian.net/package/totem-plugin-arte

The version I'm proposing is directly based on the latest upstream release, 
which fix this problem and some other (small) issues.
Backporting the relevant commits as patches to the current wheezy version would 
bring most of the code from the new upstream version, so I thought adding a 
small patch to make the new version build with the old Totem from Wheezy was 
the right thing to do.


Here is the upstream changelog since 3.1.2 (the current Wheezy version), with 
some annotations.

-

2013 September 22th - Totem Arte Plugin 3.2.1

Changes since 3.2.0:
 * Fixed the extraction of https video links
 * Fixed preferences mnemonics in English

2013 September 10th - Totem Arte Plugin 3.2.0

Changes since 3.1.3:
 * Fix stream extraction from the new Arte website
 * Reimplementation of the removed Totem.CellRenderVideo - could be reverted
 * New and more efficient primary video feed source (JSON) - Now required
 * Support for low quality streams - could be reverted
 * Minor bug fixes and translation updates

2012 December 9th - Totem Arte Plugin 3.1.3

Changes since 3.1.2:
 * Fix compilation with Vala 0.18 and Totem 3.6.0 - REVERTED, see 
debian/patches

-

The new package also includes the harden binary that is already in experimental 
and Ubuntu since many months.


Here is the proposed debian/changelog:

totem-plugin-arte (3.2.1-1~wheezy1) stable; urgency=low

  * New upstream release
  * Add a patch to build against Totem 3.0 (revert an upstream commit)
  * Harden the binary, fix 2 lintian warnings
  * Bump Debian Policy to 3.9.4 (no change needed)

 -- Nicolas Delvaux cont...@nicolas-delvaux.org  Wed, 02 Oct 2013 00:20:15 
+0200




Totem-plugin-arte is a leaf package and the current Wheezy version is unusable.
IMHO, this looks like a very low risk upgrade.


Regards,
Nicolas


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org