Bug#885074: visualboyadvance: Drop (Build-)Depends on libglademm2.4

2017-12-28 Thread Etienne Millon
On Sat, Dec 23, 2017 at 10:34:01AM -0500, Jeremy Bicha wrote:
> Your package depends and build-depends on libglademm2.4. Please drop
> that dependency. (That might require porting your app to GTK+ 3.)
> 
> Otherwise, please consider requesting that your package be removed
> from Debian to help us complete this goal.

Hi,

The proper solution will be to package vba-m, which is more up to
date in this respect.

In the meantime, I'll remove the visualboyadvance-gtk binary package
which is GTK 2.4 only. Visualboyadvance can still be used using the
commandline tool (or using other frontends).

Thanks!

-- 
Etienne Millon


signature.asc
Description: PGP signature


Bug#877103: ITP: python-rebulk -- Define simple search patterns in bulk to perform advanced matching on any string

2017-09-28 Thread Etienne Millon
Package: wnpp
Severity: wishlist
Owner: Etienne Millon 

* Package name: python-rebulk
  Version : 0.9.0
  Upstream Author : Rémi Alvergnat (https://github.com/Toilal)
* URL : https://github.com/Toilal/rebulk/
* License : MIT
  Programming Lang: Python
  Description : Define simple search patterns in bulk to perform advanced 
matching on any string

ReBulk is a python library that performs advanced searches in strings
that would be hard to implement using re module or String methods
only.

It includes some features like Patterns, Match, Rule that allows
developers to build a custom and complex string matcher using a
readable and extendable API.

This library is a dependency for new versions of python-guessit,
already in Debian and maintained by myself in the DPMT.

I intend to maintain it there as well, and comaintainers are more than
welcome.

Thanks

-- 
Etienne Millon



Bug#806034: gmpc: FTBFS when built with dpkg-buildpackage -A (No such file or directory)

2016-05-29 Thread Etienne Millon
tags 806034 + patch
thanks

Hi,

Here's a patch that fixes the issue.

Thanks!

-- 
Etienne Millon
>From b9b2535610ffeef7958dea212806b1dbca31e1bb Mon Sep 17 00:00:00 2001
From: Etienne Millon 
Date: Mon, 2 May 2016 21:47:59 +0200
Subject: [PATCH] create doc directory if it does not exist

---
 debian/changelog | 6 ++
 debian/rules | 2 ++
 2 files changed, 8 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 8d66496..785e406 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+gmpc (11.8.16-10) UNRELEASED; urgency=medium
+
+  * Fix FTBFS when built with dpkg-buildpackage -A (Closes: #806034)
+
+ -- Etienne Millon   Tue, 03 May 2016 00:01:38 +0200
+
 gmpc (11.8.16-9) unstable; urgency=medium
 
   * Fix building with a recent vala (Closes: #739030)
diff --git a/debian/rules b/debian/rules
index 753c487..771574a 100755
--- a/debian/rules
+++ b/debian/rules
@@ -20,8 +20,10 @@ override_dh_install:
 override_dh_installdocs:
 	dh_installdocs
 	make -C doc/Mallard DESTDIR=debian/tmp/usr install
+	mkdir -p debian/gmpc-dev/usr/share/doc
 	rm -rf debian/gmpc-dev/usr/share/doc/gmpc-dev
 	ln -sf gmpc debian/gmpc-dev/usr/share/doc/gmpc-dev
+	mkdir -p debian/gmpc-dbg/usr/share/doc
 	rm -rf debian/gmpc-dbg/usr/share/doc/gmpc-dbg
 	ln -sf gmpc debian/gmpc-dbg/usr/share/doc/gmpc-dbg
 
-- 
2.8.1



Bug#815204: rss2email: option to deliver to an mbox or Maildir

2016-05-29 Thread Etienne Millon
* Paul Wise  [160529 12:33]:
> I would like to switch from liferea to rss2email but I'd like to
> preserve my folder structure from liferea. To switch over I would
> need rss2email to support storing items in mboxes or Maildirs on a
> per-feed basis so that I can create the tree structure and assign
> feeds to dirs.

Hi,

While it does not directly answer your question, you can configure
esmtp+procmail to deliver to per-feed directories.

Pieces of configuration can be found in
https://github.com/wking/rss2email/pull/21 (as well as a patch for
maildir support).

Hava a nice day

-- 
Etienne Millon



Bug#823113: override: subliminal:video/optional

2016-04-30 Thread Etienne Millon
Package: ftp.debian.org
Severity: normal

Hello,

As mentioned in #818333, the correct section for the binary package
subliminal should be "video" and not "python".

Thank you!

-- 
Etienne Millon



Bug#805241: Overwrites config file

2015-11-23 Thread Etienne Millon
forwarded 805241 https://github.com/wking/rss2email/pull/73
thanks

* martin f krafft  [151116 17:16]:
> The prettify post-process hook does not work out of the box. This
> patch fixes it.

Hi Martin, thanks for the patch! I forwarded it upstream and will
include it in the next upload.

> rss2email should probably also recommend python3-bs4.

Idem, thanks for the suggestion!

-- 
Etienne Millon



Bug#805224: Overwrites config file

2015-11-23 Thread Etienne Millon
tags 805224 + upstream patch
forwarded 805224 https://github.com/wking/rss2email/pull/72
thanks

Hi Martin,

Thanks for reporting this bug. I informed upstream of this issue and
prepare a patch. I will upload a fixed version shortly.

Have a nice day!

-- 
Etienne Millon
From 6ce465dac257c29a9f43fe16fd3bafd6a383a445 Mon Sep 17 00:00:00 2001
From: Etienne Millon 
Date: Mon, 23 Nov 2015 20:45:02 +0100
Subject: [PATCH] r2e new: do not overwrite existing config
Bug-Debian: https://bugs.debian.org/805224
Forwarded: https://github.com/wking/rss2email/pull/72

If a configuration file already exists, "r2e new" will overwrite it.
This was reported in the following bug report:

https://bugs.debian.org/805224

This adds a check when running "r2e new" that will exit if the configuration
file already exists.

Signed-off-by: Etienne Millon 
---
 rss2email/command.py | 2 ++
 rss2email/error.py   | 6 ++
 rss2email/feeds.py   | 3 +++
 3 files changed, 11 insertions(+)

--- a/rss2email/command.py
+++ b/rss2email/command.py
@@ -31,6 +31,8 @@
 if args.email:
 _LOG.info('set the default target email to {}'.format(args.email))
 feeds.config['DEFAULT']['to'] = args.email
+if feeds.configfile_exists():
+raise _error.ConfigAlreadyExistsError(feeds=feeds)
 feeds.save()
 
 def email(feeds, args):
--- a/rss2email/error.py
+++ b/rss2email/error.py
@@ -258,3 +258,9 @@
 def __init__(self, **kwargs):
 message = 'error reading OPML'
 super(OPMLReadError, self).__init__(message=message, **kwargs)
+
+
+class ConfigAlreadyExistsError (FeedsError):
+def __init__(self, feeds=None):
+message = 'configuration file already exists'
+super().__init__(feeds=feeds, message=message)
--- a/rss2email/feeds.py
+++ b/rss2email/feeds.py
@@ -337,6 +337,9 @@
 'cannot convert data file from version {} to {}'.format(
 version, self.datafile_version))
 
+def configfile_exists(self):
+return _os.path.exists(self.configfiles[-1])
+
 def save(self):
 _LOG.debug('save feed configuration to {}'.format(self.configfiles[-1]))
 for feed in self:


signature.asc
Description: Digital signature


Bug#805439: RFS: visualboyadvance/1.8.0.dfsg-3 [RC]

2015-11-19 Thread Etienne Millon
* Gianfranco Costamagna  [151119 19:22]:
> I guess my upload was the first one... not sure, we will see :)

Thanks to both of you!

-- 
Etienne Millon



Bug#805439: RFS: visualboyadvance/1.8.0.dfsg-3 [RC]

2015-11-18 Thread Etienne Millon
Package: sponsorship-requests
Severity: important

(CCing Vincent Legout who sponsored previous uploads and Chris Lamb
who reported this bug - thank you guys!)

Dear mentors,

I am looking for a sponsor for my package "visualboyadvance"

 * Package name: visualboyadvance
   Version : 1.8.0.dfsg-3
   Section : otherosfs

It builds those binary packages:

visualboyadvance - full featured Game Boy Advance emulator
visualboyadvance-gtk - GTK+ front-end to VisualBoyAdvance emulator

This upload fixes RC bug #805163 (FTBFS due to a missing header).

Please note that I am a Debian Maintainer, but currently lack rights
to upload this package. So if you are interested in uploading this
package I would appreciate if you would set the correct ACL so that I
can upload this package my self.

The latest changelog entry is:

visualboyadvance (1.8.0.dfsg-3) unstable; urgency=medium

  * Add a patch to fix CPU usage under SDL.
Thanks to Matthew Harvey (Closes: #740292)
  * Change my email address.
  * Bump S-V to 3.9.6 (no changes needed).
  * Remove menu file.
  * Remove obsolete include . (Closes: #805163)

I uploaded the package to mentors at the following URL:

dget -x 
http://mentors.debian.net/debian/pool/main/v/visualboyadvance/visualboyadvance_1.8.0.dfsg-3.dsc

It is also available on the Debian Games Team git repository:

git://anonscm.debian.org/pkg-games/visualboyadvance.git

Thanks!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#796924: lintian: False positive for source-is-missing and python3

2015-08-25 Thread Etienne Millon
Package: lintian
Version: 2.5.36.1
Severity: normal

Hello,

I'm currently preparing guessit 0.10.4-1, not uploaded yet but
available in the Python Modules Team's svn repository at:

svn://anonscm.debian.org/python-modules/packages/guessit/trunk

Lintian reports the "source-is-missing" error for
docs/_themes/__pycache__/flask_theme_support.cpython-34.pyc.

However, the source of this file is available in
docs/_themes/flask_theme_support.py.

In python3, the pyc files are in a __pycache__ file and their
filenames include the version of the compiler ("cpython-34" here). I
believe that this mechanism is not understood by lintian, which causes
this false positive. Most information about it can be found in the
following document:

https://www.python.org/dev/peps/pep-3147/

Please let me know if you need more information.

Thanks for the awesome work you're doing on Lintian!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#792420: zsnes: emulator escape vulnerability

2015-07-14 Thread Etienne Millon
* Paul Wise  [150714 18:20]:
> According to this Youtube video and forum post, there are at least 3
> vulnerabilities in zsnes that allow ROMs to escape the zsnes
> emulator and execute arbitrary code on the host running zsnes. The
> known issues will be fixed in 1.52 but there may be more issues.
> This may or may not be related to the cppcheck warnings from bug
> #610313.

Thanks for the report.

While neither the exploit code nor a fix is out, I believe that the
best course of action is indeed to write a patch for #610313.

It may also be possible that due to hardening patches, this bug is not
exploitable in Debian.

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#784788: rss2email: r2e-migrate forgets about email addresses

2015-07-04 Thread Etienne Millon
tags 784788 + patch
thanks

* Yves-Alexis Perez  [150510 17:28]:
> I just upgraded rss2email from 2.x to 3.x, so the config file format
> has changed. I tried to migrate my configuration, but unfortunately
> the per-feed email address (which I'm using intensively) was not
> migrated. This is really painful, especially considering it's
> apparently not possible to edit a feed afterwards.

Hi Yves-Alexis,

I prepared a patch which should add your feeds with the correct email
address. I'll include it in my next upload.

Thanks for the report!

-- 
Etienne Millon
From 170bbb4516c3381450cf2fedd0e021a7cef847ef Mon Sep 17 00:00:00 2001
From: Etienne Millon 
Date: Sat, 4 Jul 2015 12:30:20 +0200
Subject: [PATCH] r2e-migrate: support per-feed addresses

---
 debian/changelog   |  1 +
 debian/r2e-migrate | 12 +---
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 98a63bf..4f6796c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,7 @@ rss2email (1:3.9-2) UNRELEASED; urgency=medium
 
   * Bump Standards-Version to 3.9.6 (no changes needed)
   * Use TLSv1 by default instead of SSLv3 (Closes: #781315)
+  * r2e-migrate: add per-feed addresses (Closes: #784788)
 
  -- Etienne Millon   Sat, 04 Jul 2015 11:59:25 +0200
 
diff --git a/debian/r2e-migrate b/debian/r2e-migrate
index 47844cb..393d067 100755
--- a/debian/r2e-migrate
+++ b/debian/r2e-migrate
@@ -10,6 +10,9 @@
 #
 # Changelog:
 #
+#   v5 (2015-07-04)
+#  - support per-feed addresses
+#
 #   v4 (2014-06-10)
 #  - support XDG directories
 #
@@ -81,8 +84,11 @@ def slugify(s):
 return ''.join([slugify_char(c) for c in s])
 
 
-def add(url, name):
-return subprocess.call(['r2e', 'add', name, url])
+def add(url, name, to):
+extra_args = []
+if to is not None:
+extra_args = [to]
+return subprocess.call(['r2e', 'add', name, url] + extra_args)
 
 
 def pause(name):
@@ -118,7 +124,7 @@ def main():
 url = feed.url
 print url
 name = slugify(url)
-add(url, name)
+add(url, name, feed.to)
 if not feed.active:
 pause(name)
 modified = None
-- 
2.1.4



signature.asc
Description: Digital signature


Bug#781315: Wrong default in the config -- unusable with SMTP after a recent Python upgrade

2015-07-04 Thread Etienne Millon
* Andrew Shadura  [150704 12:23]:
> Hi,
> 
> Probably your Python is too old :)

My bad, that was on a jessie box...

I'll fix that, thanks!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#781315: Wrong default in the config -- unusable with SMTP after a recent Python upgrade

2015-07-04 Thread Etienne Millon
Hi,

Thanks for your bug report. I replaced the default value in the
configuration file and will add a warning at runtime, but I don't have
a missing attribute error here:

% python3
Python 3.4.2 (default, Oct  8 2014, 13:14:40) 
[GCC 4.9.1] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import ssl
>>> ssl.PROTOCOL_SSLv3
1

Is it something that can be missing on different platforms?

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#787859: rss2email: Most links wrap

2015-06-29 Thread Etienne Millon
title 787859 Configure html2text using keyword arguments
thanks

Hi,

Thanks for your bug report.

It seems that there are two bugs here:

  - rss2email does not configure the correct BODY_WIDTH in html2text.
  - in html2text, there is no way to configure a different wrapping
policy for text and links.

The second is #616090. The first one is due to the fact that html2text
now needs to be configured using keyword arguments instead of globals.

Reading the code, it seems that the old way using globals is intended
to be supported, but default arguments are evaluated at import time,
so it's impossible to configure html2text by overriding
html2text.config.

As for your problem, once the rss2email bug (or the html2text config
one) is fixed, you should be able to have long lines by setting
body-width=0 in rss2email's configuration.

-- 
Etienne Millon


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



Bug#790202: gmpc-plugins: depends on libwebkitgtk-1.0-0 which is deprecated

2015-06-28 Thread Etienne Millon
tags 790202 + patch
thanks

Hi,

Thanks for the bug report.

Upstream is not very active on this project and I don't have a lot of
experience porting GTK applications. So I think that the best course
of action is to disable the only plugin that uses webkitgtk.

However, if anyone wants to do the actual porting, I am interested of
course!

-- 
Etienne Millon
commit 7b0a45d96d231f3f0368756bbdd842d7305b15ca
Author: Etienne Millon 
Date:   Sun Jun 28 19:00:52 2015 +0200

Disable wikipedia plugin

diff --git a/debian/changelog b/debian/changelog
index c744dc7..0db003b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+gmpc-plugins (11.8.16-3) UNRELEASED; urgency=medium
+
+  * Disable wikipedia plugin (Closes: #790202)
+
+ -- Etienne Millon   Sun, 28 Jun 2015 18:59:43 +0200
+
 gmpc-plugins (11.8.16-2) unstable; urgency=low
 
   * Disable LyricWiki plugin (Closes: #539821)
diff --git a/debian/control b/debian/control
index ceb2177..9d61b8e 100644
--- a/debian/control
+++ b/debian/control
@@ -20,10 +20,8 @@ Build-Depends:
  liblircclient-dev,
  libmicrohttpd-dev,
  libmpd-dev,
- libsoup2.4-dev,
  libsqlite3-dev,
  libtagc0-dev,
- libwebkitgtk-dev,
  libxml2-dev,
  libxosd-dev,
  valac
diff --git a/debian/rules b/debian/rules
index 5797a8a..28460b8 100755
--- a/debian/rules
+++ b/debian/rules
@@ -9,7 +9,7 @@ DEB_DH_STRIP_ARGS:=--dbg-package=$(DBG_NAME)
 	dh $@ --with autoreconf
 
 override_dh_auto_configure:
-	dh_auto_configure -- --disable-lyricwiki
+	dh_auto_configure -- --disable-lyricwiki --disable-wikipedia
 
 override_dh_strip:
 	dh_strip $(DEB_DH_STRIP_ARGS)


signature.asc
Description: Digital signature


Bug#787223: RFS: vbam/1.8.0.1498-1

2015-06-16 Thread Etienne Millon
Hi there! Thanks for reaching out.

The Games Team is probably the best place to maintain this package
indeed. I myself maintain two console emulators there: zsnes and vba.

VBA-M is a good addition, way more recent than VBA. It's more
maintained and more accurate:

http://tasvideos.org/EmulatorResources/GBAccuracyTests.html

It is not entirely clear whether it should replace VBA. What do you
think? If it is so, it will probably be worth making an upgrade path
so that existing vba installs are converted to vba-m (this does not
have to be in your initial upload).

I mostly agree with the first review, so I won't go in details over
this.

> > * You probably want a versioned dependency from vbam-gtk and vbam
> > -wx to vbam-common.
> 
> vbam-common has only icons, so I don't understand the need for this.

This is needed for upgrades. It makes it impossible to have vbam-sdl
1.9 installed together with vbam-common 1.8.

> > W: vbam source: missing-license-paragraph-in-dep5-copyright
> > public-domain
> (paragraph at line 36)
> 
> I don't understand this warning, "public-domain" isself explanatory
> and it's not a license at all. I'm trying to understand the text
> below from the copyright-format page, but I'm not fluent in the
> english language. They should add an example to clarify:
> 
> "When the License field in a paragraph has the short name
> public-domain, the remaining lines of the field must explain exactly
> what exemption the corresponding files for that paragraph have from
> default copyright restrictions."

"Public domain" is unfortunately ambiguous, and in some countries it
does not even exist. That is why it is need to be clarified.

Generally speaking, as an author it is a good idea to use CC0 which
explicitly describes these rights.

In your case, adding a short description "These files have been put in
the public domain by their author" may be enough.

Feel free to CC me in your replies and I'll be happy to make the next
review once those issues are fixed!

And thanks for working on VBA-M!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#781622: RFS: opentyrian/2.1.20130907-1 [ITP]

2015-04-02 Thread Etienne Millon
* Dmitry Smirnov  [150402 08:25]:
> On Wed, 1 Apr 2015 09:20:34 Etienne Millon wrote:
> > I refreshed this, forwarded two patches and picked the upstream
> > version of one.
> 
> Nice. But I meant to use all those headers (except for Applied-Upstream which 
> I use ocassionally).

Oh, I see. I added them.

> That's good but let's repack orig.tar to remove this file unless you
> want to wait till next release. Ftp-masterd do not like blobs and
> I'm not sure if they will be willing to tolerate this particular
> one. Better not take chances and not waste their time.

Thanks for the heads-up on the new way of repacking. Files-Excluded
makes it real easy indeed.

> Speaking about watch file I recommend to extend regex to match other types of 
> archives, something like 
> 
> opentyrian-(.*)-src\.tar\.(?:gz|bz2|xz)

Done.

> Thanks. I think same issue exists in copyright of Andrea Mazzoleni.
> Could you fix it too please?

Done, repushed, reuploaded!

-- 
Etienne Millon


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



Bug#781622: RFS: opentyrian/2.1.20130907-1 [ITP]

2015-04-01 Thread Etienne Millon
* Alexandre Detiste  [150401 20:22]:
> Le mercredi 1 avril 2015, 20:38:30 Dmitry Smirnov a écrit :
> > Hmm, how about doing it gnu-make-style?
>  
> I tried that too, but this doesn't work as I expected ("Only the first ‘%’ in 
> the pattern and replacement is treated this way; any subsequent ‘%’ is 
> unchanged." from the manual)
> 
> --
> 
> icons := $(patsubst 
> linux/icons/tyrian-%.png,debian/opentyrian/usr/share/icons/hicolor/%x%/apps/opentyrian.png,$(wildcard
>  linux/icons/tyrian-*.png))
> 
> debian/opentyrian/usr/share/icons/hicolor/%x%/apps/opentyrian.png: 
> linux/icons/tyrian-%.png
> install -D --mode=644 --preserve-timestamps $< $@
> 
> override_dh_install: $(icons)
> dh_install
> 
> 
> > Please note that in make files "make" is usually spelled as "$(MAKE)".
> Oops, I overloked that.
> I had to add even more boilerplate to make parallel build actually work.

I reuploaded the package to mentors with changes related to
reproducible and parallel builds.

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#781622: RFS: opentyrian/2.1.20130907-1 [ITP]

2015-04-01 Thread Etienne Millon
* Dmitry Smirnov  [150401 07:34]:
> On Tue, 31 Mar 2015 19:35:12 Etienne Millon wrote:
> > I am looking for a sponsor for my package "opentyrian"
> 
> That is very nice. I was looking forward to see opentyrian in Debian for a 
> while. Thanks for packaging.

Thanks for taking the time to look at this.

> > I'm happy to hear your remarks about this package.
> 
> First thing to improve is to add more DEP-3 [1] headers to patches.
> [...]
> If you did not forward patches yet I'd recommend to wait no further and 
> document progress as described.

I refreshed this, forwarded two patches and picked the upstream
version of one.

> Also there are some remarks about packaging:
> 
>   * There should be versioned Depends on "game-data-packager" which
>   actually support "tyrian-data" (i.e. "tyrian-data |
>   game-data-packager (>= 40)").

Done.

>   * Package should install icon (there are some in "linux/icons).
>   Icon is referenced from installed .desktop file.

I'm not too familiar with how icons work, so I've installed them as
/usr/share/icons/hicolor/NxN/apps/opentyrian.png. Is that correct?

>   * Repository do not match package uploaded to Mentors. There are
>   differences in "control" (Standards-Version) and in README.Debian.

Classic case of forgotten push. Fixed.

>   * Why not enable full hardening?
> (e.g. "export DEB_BUILD_MAINT_OPTIONS = hardening=+all")

Done!

>   * Re-distribution of pre-built binary "macosx/tyrian.icns" in
>   source archive may be a bit of concern.

It's being removed in the next release:

https://bitbucket.org/opentyrian/opentyrian/commits/e4346769d72374e51f34eecf45515b6df85d00c0

>   * There is a comma "," which is not present in the original copyright 
> statement after copyright year in 
> 
> Files: ./src/video_scale_hqNx.c
> Copyright: 2003, MaxSt ( ma...@hiend3d.com )
> 
> IMHO it should be just "2003", not "2003,".
> Other than this "debian/copyright" looks good.

Indeed, fixed that.

I repushed and reuploaded the package.

Thanks!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#781622: RFS: opentyrian/2.1.20130907-1 [ITP]

2015-03-31 Thread Etienne Millon
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package "opentyrian"

 * Package name: opentyrian
   Version : 2.1.20130907-1
   Upstream Author : The OpenTyrian Development Team
 * URL : https://bitbucket.org/opentyrian/opentyrian/wiki/Home
 * License : GPL-2+
   Section : games

It builds those binary packages:

opentyrian - open-source port of the DOS shoot-em-up Tyrian

OpenTyrian (ITP #553035) is a free engine for non-free game tyrian,
whose assets are downloaded by game-data-packager.

I uploaded a package on mentors and it is dgettable at the following
URL:

dget -x 
http://mentors.debian.net/debian/pool/contrib/o/opentyrian/opentyrian_2.1.20130907-1.dsc

Alternatively, you may access the git repository at:

git+ssh://git.debian.org/git/pkg-games/opentyrian.git

I'm happy to hear your remarks about this package. Also, please note
that I'm a Debian Maintainer, so once this clears NEW I'm interested
in uploading the next revisions myself. Thanks!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#724718: ITP: flexget

2015-01-25 Thread Etienne Millon
See #688551 for background on python-dateutil. Basically, versions >=2
used to be a python3 only fork, but now that it has been merged back
it should be compatible (removing the upper bound should work).

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#775609: Remove Python 3 version of module to fix RC bug? (python-exif)

2015-01-25 Thread Etienne Millon
Hi,

To remove the binary package two things are needed:

  - do not build the binary package anymore. Removing it from
d/control does the trick, but will leave a lot of cruft from the
source package. For example, d/python3-exif.install will be unused
as well as the sed line in d/rules. You will also need to remove
python3-* from Build-deps.

  - remove the leftover binary package. In this case this should be
automatic: auto-cruft should detect that python3-exif is NBS (Not
Built from Source) and this should automatically propagate to
testing (as there are no reverse dependencies for python3-exif).
You can have a look at https://wiki.debian.org/ftpmaster_Removals
for more info.

Cheers

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#535073: ITP: dolphin-emu -- Dolphin Gamecube / Wii Emulator

2015-01-11 Thread Etienne Millon
Hi James.

Thanks for working on dolphin, it will be a great addition to the set
of emulators available in Debian.

I had a look at your package and it looks very good to me (although I
have not had a thorough look at d/copyright). The only remark I have
is that you can drop the version constraints on cmake and libxi-dev
which are satisfied in oldstable ("cme check dpkg" returns this
information).

Cheers

-- 
Etienne Millon


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



Bug#774708: debian-maintainers: Please add Etienne Millon as a Debian Maintainer

2015-01-06 Thread Etienne Millon
Package: debian-maintainers
Severity: normal

Dear Maintainers,

Please add Etienne Millon  to the Debian Maintainers
keyring. Please find the jetstring changeset attached.

Regards,

-- 
Etienne Millon
Comment: Add Etienne Millon  as a Debian Maintainer
Date: Tue, 06 Jan 2015 16:31:30 +0100
Action: import
Recommended-By: 
  Amaya Rodrigo Sastre 
  Vincent Fourmond 
Agreement: 
  https://lists.debian.org/debian-newmaint/2014/10/msg00023.html
Advocates: 
  https://lists.debian.org/debian-newmaint/2014/10/msg00027.html
  https://lists.debian.org/debian-newmaint/2014/10/msg00040.html
Data: 
  -BEGIN PGP PUBLIC KEY BLOCK-
  Version: GnuPG v1
  
  mQINBE+ulTUBEACvSnHXl9ByEbOVgFHUVv05IpJPSrrpQWIAZvCDBE6pHUeWR6KO
  2K5MMJ292zbCkw+XJgRqU5XE9rorqna9MfS3UfLTmpY/nBvxUYAhfN8jWWnM714c
  uq46zn9f4/xHLRp/IwJlxasBgpYemAeKTQrxO0QfDEIT+y3XQqFuqJ10tEPNTH85
  erhf+HSxZLTm7uCNDvEVqePEpneAA+f8Ub3/bLmOvuSWPRRVxJ3+0pFC6q5wKI71
  FNxjCiOyJ8j6a3+pYGJGoDdeibHEoKtkIJhQxMN+HOh2EnzsohjMQhpsUFqaI7Qh
  jaCyDk7rs8f5nEk4+88DlDAKvWbOGUBx0HCbqg9ILx+oC8/yMkP9fhKCD1s0oV0m
  LU0RxIPtpo+Gnv9Ojy2dgk+ZeSgaK5fReyTivbZ8ioxUbtCWy/A3v9C7Ko55GnX3
  WocgRQU3dvsqu97Oj2MmR6+kSlkCqFOy50tzh9Ptah5Mhk/bzhOvFxrU6IZ50neP
  DwyqI8haRmZoe40EBy3yhXxXVFTvlh9Ute0HZ1qI2VcjTgKBx+3mZLTlz4/AE0Xf
  aJEUGY1yaFpQ+kTUl6kpQu54jzboR8N/ZxB2j7J23McMxGd4nlbnzd5v9fTP/hF8
  Ze1lCtNyhAwRWNkMGsIo2D5HPteXXLQRI6MD7fnxpTrgbE6sZBs0QWPslQARAQAB
  tB9FdGllbm5lIE1pbGxvbiA8bWVAZW1pbGxvbi5vcmc+iQJABBMBCAAqAhsDBQkF
  o5qAAh4BAheAAhkBBQJPrppABQsJCAcDBRUKCQgLBRYCAwEAAAoJEJhm5KnsFmPl
  Xv0QAIGPQp2jDUydQ0b6aT7Fehfhv/XdZLnmEQ3myiSwq8YQoZ8ilSHwJSthpmKb
  PqIbJl07Y5FgmJ9Py61wDvKjzKN8F8wnH3S2UGIkR4d8kUVV6bMCvOI7X6ErE70Q
  /PVIaRTcBWv23O0+iNlD4jO6AfnbfoDjtwOnql8LCt0Mk5IhtN9xaRlpmaDdr7xB
  ZF4JhYNDN+ietB4N1yUPOJVb/YOOK/B05hS+yj2xA+eHXlyUyXaeuAOJe7teK04D
  98ik7qaPTUDGT25lZKAYrQFxrqKO0GiPSpoq4X4VwYDoxwBLN69LnssDhOQ1xYW1
  /2cBdy+KlIqmABXeN6Blf9ano0ffwQJ/k1dibGeotvbwfsLOjxAwr/08oMJbAHaH
  pLq7Z0waVEPwxSg/G/Bc28DVa5oR2PKgTKwXSVVQ3XYMfDSeaEY8JW0k+9gvTwQl
  LCffQc4xFak7wwg88H9ZE8q+N39dqpyzUSBXm/ecsAWwW3Gnnb5cZAuE7AOmunig
  9taRRCktpv413KgP2i0RzjnwvAREn98GIgsicZwzW52BAobOLd5uPG5UBLpeMlnW
  9A07eXd1UWkdVZgj0ftHspd9svC+9gSbv9d4O5NWCTA3DlNWDki4oLzarXGtwWod
  jY0/tYv3rEGmkeJpQb2WPscfsTKj03v4JgB2QQxtyGIaobnuiQEcBBABCAAGBQJP
  rpq5AAoJEObsMVOU5V5VBtAH/jZ9eVDmw9/XFs12nfs3ToTt79ruCp2rXxcvdhOB
  VUG46DlmpIBWzmN4bnaUT1gE2FBXRy3TQynNC03fm7bg9x1OAc13oRNgx8qnwoiU
  UmK79sgtaa3KGImZfJi4hVrDeFaieZkJLZ9JqkBekmu2l948D/A8Wc7g9nBpPcuD
  6Ert7McSq/eyukAUfMoeBM9VKvPRFij/b5j+DzkQgsCehVIF70Je2Xa4n1nORKRy
  ljEWCMU+LAjgDHDEnEhD3vcfy4xK6KN644YLPRvO+1HlxDt+rnwRCcoaZjR1+MPW
  p5jPk+sOoyjTGT6BfPFPZVQJsZBw7xM0W8WSVBbWQ9Tp1qGJAhwEEAECAAYFAk+6
  vjwACgkQRSm8cJju03nyzw//TU6fuH1nod6hUkFJIZZFFql/xozOQx4MqEXDVG4v
  qKUy84d/AhdoWQ/VPNhPlPstvhHTchoJQ5DpHTLmZZVtXDmAvKriEL9YlWrtzovC
  N4RyJygl/wGI6MYeY9Bm8f9T955HGJhYvlQoL4ngpQImLbUsTo4rXvOD0Bc+2/Xh
  0z+jtYvisqTYwf1MqllqNC7LFTBeUOf9rGKCKsP3m+Ocd+O+X5gVY3XWF3mM8LwT
  H6KsoI+7f+9a2aap7fzdT+OLy5XYBmPqoZg7C4dOR2AdligMNn3JeK+gNx0xGCJR
  +PCkX53KHrEgQkDUPcZnlPlxR07l3UnQl67WnW6TGQlPVvtiI/ozW+PDUG4L/sRS
  9IGMN9qCXcQNNq0jD4KLkLnYf4bEsFTeFPqIL5NGHXTEtOMx64egap22ihDYxW7r
  ZZP0sDmZgs3Oadx9iZRb5ZOn16QJvLdiMUoqtDUW9qD8vPvbXayCK4W+FHf6QX2P
  jfbCVMTiElV8eNkYx73SsWsCP7BE/RIgiIgJBr28Rcv82fSIacf3zVWSfT4UEuSK
  luSRNMyKuabEXnknylXo/LR4k8gu6GKWmkV1CMnWJISqwl0/+iZwPf4FA5eq5PtK
  jvovhg2rfHwYbt4+Mb9C4rqodzpvXGuVbP75taevVP4PPy0tAwsWMKoMbXU0fCfQ
  K1OJAiAEEwEKAAoFAk+6y/gDBQE8AAoJENbvpqxLENhHLu4P/jVZZPgJ5JvJdaaJ
  CbO765y8Puzr2Wn6TS6PiYlYKxu6c72kZIDGrRaMfpgnH//sxIy0SGcrZ7oLKorf
  APbcgFFC/igxb6vphuMrUB0kE9oFWS5CR4m00G6ZNGN4Wgx3pyJY220iTgZR8HTX
  f9pUWXNtcDHMqxcvXn4cjQLEGZQIxt+cHS7/6N+Unj/WZMyBu55j0ya3IBNkyKkz
  fxFfCwoZ0lrhTjiHsChpHnfnrjUzNifPUcNVwafPxcfW2i5uLXCHCXNdBQnSukVL
  PZye2I8mOigym/vvYMsenGXLwavSPXfnYXJ76XwwzCM0lUNXNovO+tlgPSF+owVy
  HPPCjo2Ltm7Bz+cXAYL2XhXCuOP3JRUf7EGcAQ597xY4IHLD1XdolPqkOt3+NghW
  q4q7I8FqGL7yHIdx4zCeM7Rosr4a0DaWa0Q4HzUqE9DN3gEnJojhooBgr08ghAxp
  Dz1CEyKHZlk5rGFkg1J13YJjPbTPn1N4IZIDnDE04O9dAQDsaHLnI3AIMLwURjqK
  dMY4dPsWz8dR42PoR4qK6S+fT1OccRG9vi4lDD6C732y2jsjSFgkiPyyWLORgMgQ
  FBcoh/FcV1HMcDtnEjnKx80gzeUEikqqJpWhNU3dFhgd8ssw5KYP+uO+4Rb70ufn
  vFNBR0SZFW/yN0+nPIODGpWMHvH8iEYEEBECAAYFAk+6010ACgkQW1gkPL5QA1Z/
  FACfSSlkxJcGQcsv06iBgNfdcNz5aYMAnRi1R+iGI/I7ojvHNjii++fluej7iEYE
  EBECAAYFAk+62SsACgkQ4HMF45zERF5ZRgCfYEIO2C9IBHV4oAkDyjG91JVwqOMA
  njyIEom6lWl7ukJzXs2AmwVPw/8xiQIcBBABCAAGBQJPurSCAAoJEOptTrkAMhVV
  qIIP/0ke/qm+4H4p7ztBAZJe7TPKLREcq4MGF8/LJhGODvA0DtgnfmJ5nu9AolMx
  4KVay7nT/gLRtdTFlhsWYKCzahqS8zXxnzTpdTGN5pA3IV+57jrKYoQXmlJdvJn0
  dULirii07NlwbPOWSi2KqBUcZSo+LcuiOoFlkx43c63+s4dmXtSbmmdkE6D3MTDD
  o8rCjJpb4XiBJAV3Er/xANtBKTWDBQnxhVQ2PqCdQpZxyUbITmgk6M4YD3cD3LuV
  92yFQ8xAdQVPgriwy6HugGtGo50jgOTBU0XVKuLzZtfZn2Bb8unyMaOAB+sCym48
  kKPCbUN19tLCpuIkCEySoq7ycXTpoCOegDETnrVpQOEzzRAL5e6IAjGeg4H6SOI5
  eV7pfhzzOC+GgCo9pa6SyvUxnfjaojy2SL6mH483Xw/eiDtqRzxJV2qfhX6kJZu6
  OqJO7LQVhvkcAHqe7kOPw40fktlGNEqLkcyTKgqDjAJHvos2CYuYtTXeIUU0Q/bu
  oLuHtFpd+zWLJiAvhwyRwlDvETsimYUQy45A1maGzWVGZBUP/BK

Bug#773810: RFS: averell/1.0a-1 (ITP: 773793)

2014-12-23 Thread Etienne Millon
Hi Jean,

I had a quick look at your package.

  - the code uses embedded dependencies. Please use packaged ones
instead of these embedded copies (i.e. build-depend on
erlang-cowboy). If some are not properly packaged in Debian, you
will have to package them first.

  - your package fails to build in a clean chroot. The relevant build
log lines are:

> make[3]: Entering directory '/tmp/buildd/averell-1.0a/deps/getopt'
> Uncaught error in rebar_core: {'EXIT',
>{undef,
> [{crypto,start,[],[]},
>  {rebar,run_aux,1,
>   [{file,"src/rebar.erl"},{line,94}]},
>  {rebar,main,1,
>   [{file,"src/rebar.erl"},{line,49}]},
>  {escript,run,2,
>   [{file,"escript.erl"},{line,752}]},
>  {escript,start,1,
>   [{file,"escript.erl"},{line,276}]},
>  {init,start_it,1,[]},
>  {init,start_em,1,[]}]}}
> Makefile:18: recipe for target 'compile' failed

  - the long description has an extra empty line.

  - the latest Standards-Version is 3.9.6, please update accordingly.

Cheers

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#773381: subliminal: sort providers to make the build reproducible

2014-12-17 Thread Etienne Millon
Package: subliminal
Version: 0.7.4-1
Severity: wishlist
Tags: newcomer

Hi,

Subliminal's build is currently not reproducible:

https://reproducible.debian.net/userContent/rb-pkg/subliminal.html

The reason is that the manpage is generated with help2man and that the
provider list in help does not appear in the same order at each run:

% subliminal --help|grep 'providers to'
providers to use (tvsubtitles, addic7ed,
% subliminal --help|grep 'providers to'
providers to use (opensubtitles, podnapisi, addic7ed,
% subliminal --help|grep 'providers to'
providers to use (podnapisi, tvsubtitles, addic7ed,

Sorting the list should do the trick.

I added the newcomer tag, so I am happy to mentor someone in the
process of fixing this bug.

-- System Information:
Debian Release: 8.0
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.16.0-4-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages subliminal depends on:
ii  python3 3.4.2-2
ii  python3-subliminal  0.7.4-1

subliminal recommends no packages.

subliminal suggests no packages.

-- no debconf information

-- 
Etienne Millon


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



Bug#674916: s3cmd: changing from ITA to O

2014-12-03 Thread Etienne Millon
* Nicholas Robinson-Wall  [141202 09:07]:
> Jackson - you mentioned this might be better suited being maintained
> by  the python apps team. Is this something one can participate in
> without   being a full DD? I have another package which was
> sponsored into Debian, but not as part of a team. 
>   

You can absolutely be part of PAPT without being a DD. Finding
sponsors is in my experience way is easier as part of a team.
Here is the relevant pointers:

https://wiki.debian.org/Teams/PythonAppsPackagingTeam

-- 
Etienne Millon


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



Bug#769542: rss2email: option for disabling certificate verification

2014-11-19 Thread Etienne Millon
* Jakub Wilk  [141114 14:17]:
> python3.4 has recently started to verify TLS certificates by default:
> https://bugs.python.org/issue22417
> 
> Please add an option to disable certificate verification.

Thanks for the hint.

I'll probably do this after the freeze, so here is a little brain
dump:

  - this is needed for python >= 3.4.3.
  - rss2email does not make the requests itself, it just passes a URL
to feedparser.
  - on the feedparser side, the only extension mechanism is to pass a
"handlers" kwarg to feedparser.parse(). It's then passed to
urllib2.build_opener(). There does not seem to be a way to pass a
context parameter to urllib2 there.

The python bug tracker suggests to monkeypatch the ssl module to
disable verification:

ssl._create_default_https_context = ssl._create_unverified_context

It's a bit hacky, but seems simpler than patching feedparser to accept
a context argument (I would argue that fetching feeds is not
feedparser's job anyway). I would go with that solution.

-- 
Etienne Millon


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



Bug#727781: zsnes: Fatal error when loading state (DSP-1 ROM)

2014-11-17 Thread Etienne Millon
* Jason Pleau  [14 22:34]:
> Hello !
> 
> I've been trying to find a way to fix this issue as well for a few days now.
> 
> I managed to get it to work with this following changes (see attached file)
> 
> The rename of usr/bin/zsnesd => usr/bin/zsnes could probably be handled
> better (maybe with dh_links ?), for now as long as I can load my game
> states I'm happy :)
> 
> Thoughts on this patch? I'm not sure how adding --enable-debug fixes
> this, so it could very well lead to other issues.

Hi,

Thanks indeed for work! It's indeed the -O0 that fixes the issue in
that case. It's also possible to disable hardening while keeping
optimizations.

I didn't publish my patch when I worked on this last year, but here it
is.

It's tempting to apply the following hunk to continue the job and
eliminate the last copy_func call:

> -copy_func(&buffer, &Op02FX, 11*4+3*4+28*8);
> +COPY_SHORT(Op02FX);
> +COPY_SHORT(Op02FY);
> +COPY_SHORT(Op02FZ);
> +COPY_SHORT(Op02LFE);
> +COPY_SHORT(Op02LES);
> +COPY_SHORT(Op02AAS);
> +COPY_SHORT(Op02AZS);

but unfortunately it makes it crash again and I'm not sure I
understand why (especially one year later).

-- 
Etienne Millon
Author: Etienne Millon 
Description: Fix loading for DSP1 ROMS with _FORTIFY_SOURCE
 The original code uses a memory copy scheme that is unsafe and triggers
 fatal errors under _FORTIFY_SOURCE. More precisely, the scope of this
 patch is ROMs that use the DSP-1 coprocessor.
 .
 The code copies data by assuming that global variables are contiguous
 in memory: it takes the address of the first variable and uses it to
 copy several ones.
 .
 For variables declared in assembly, a new symbol DSP1state is added so
 that memory can be copied as an array.
 .
 For variables declared in C, it is trickier as it's not possible to
 alias them with an array. So a macro is used to copy variables of type
 short.
 .
 There is padding at several places:
   - 2 bytes after each short
   - 128 bytes after each "block" of variables
   - extra blocks of 236, 112 and 84 bytes
 .
 The purpose of the last ones (declared with "PADn") is unknown,
 probably other global variables that have been moved in the meantime.
 .
 In order to preserve compatibility with existing state files, this
 padding is kept. Data read from these offsets is discarded when loading
 state, and zeroes are written when saving state.
Bug-Debian: http://bugs.debian.org/727781
Last-Update: 2013-10-27
---
--- a/src/chips/dsp1proc.asm
+++ b/src/chips/dsp1proc.asm
@@ -307,6 +307,7 @@
 ret
 
 SECTION .bss
+NEWSYM DSP1state
 NEWSYM DSP1COp,   resb 1
 NEWSYM DSP1RLeft, resb 1
 NEWSYM DSP1WLeft, resb 1
--- a/src/gblvars.h
+++ b/src/gblvars.h
@@ -35,7 +35,7 @@
 extern unsigned int tempesi, tempedi, tempedx, tempebp;
 extern unsigned int SPCMultA, PHnum2writespc7110reg, PHdspsave2;
 extern unsigned char sndrot[], SPCRAM[65472], DSPMem[256], SA1Status, *SA1RAMArea, *SPCState;
-extern unsigned char DSP1Enable, DSP1COp, prevoamptr, BRRBuffer[], *romdata;
+extern unsigned char DSP1Enable, DSP1state[70], DSP1COp, prevoamptr, BRRBuffer[], *romdata;
 extern unsigned char curcyc, echoon0[], spcnumread, NextLineCache, HIRQNextExe;
 extern unsigned char vidmemch4[4096], vidmemch8[4096], vidmemch2[4096];
 
--- a/src/zstate.c
+++ b/src/zstate.c
@@ -186,22 +186,126 @@
 
   if (DSP1Enable && (method != csm_load_zst_old))
   {
-copy_func(&buffer, &DSP1COp, 70+128);
-copy_func(&buffer, &Op00Multiplicand, 3*4+128);
-copy_func(&buffer, &Op10Coefficient, 4*4+128);
-copy_func(&buffer, &Op04Angle, 4*4+128);
-copy_func(&buffer, &Op08X, 5*4+128);
-copy_func(&buffer, &Op18X, 5*4+128);
-copy_func(&buffer, &Op28X, 4*4+128);
-copy_func(&buffer, &Op0CA, 5*4+128);
-copy_func(&buffer, &Op02FX, 11*4+3*4+28*8+128);
-copy_func(&buffer, &Op0AVS, 5*4+14*8+128);
-copy_func(&buffer, &Op06X, 6*4+10*8+4+128);
-copy_func(&buffer, &Op01m, 4*4+128);
-copy_func(&buffer, &Op0DX, 6*4+128);
-copy_func(&buffer, &Op03F, 6*4+128);
-copy_func(&buffer, &Op14Zr, 9*4+128);
-copy_func(&buffer, &Op0EH, 4*4+128);
+unsigned char padding[128];
+memset(padding, 0, 128);
+#define COPY_SHORT(X) do {\
+extern short X;\
+copy_func(&buffer, &(X), sizeof(short));\
+copy_func(&buffer, padding, 2);\
+} while(0)
+#define PADn(N) do {\
+copy_func(&buffer, padding, N);\
+} while(0)
+#define PAD PADn(128)
+
+copy_func(&buffer, DSP1state, 70);
+PAD;
+
+COPY_SHORT(Op00Multiplicand);
+COPY_SHORT(Op00Multiplier);
+COPY_SHORT(Op00Result);
+PAD;
+
+COPY_SHORT(Op10Coefficient);
+COPY_SHORT(Op10Exponent);
+COPY_SHORT(Op10CoefficientR);
+  

Bug#200715: I'm still on these packages

2014-11-03 Thread Etienne Millon
retitle 200715 ITP: stepmania -- rythm and dancing game
owner 200715 !
retitle 698843 ITP: brogue -- Roguelike game by Brian Walker
owner 698843 !
retitle 704587 ITP: libtcod -- an API for roguelike developpers
owner 704587 !
thanks

Hi,

Small update. I've not been able to make good enough packages for
these packages on Jessie, but I'm still working on these ones.

Actually the freeze may be a good time to polish them so I'm claiming
again these ITPs. Of course if anyone wants to join efforts I'm happy
to have comaintainers.

Cheers

-- 
Etienne Millon


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



Bug#765074: Confirming this issue

2014-10-28 Thread Etienne Millon
tag 765074 + patch
thanks

Hi,

The problem is that vagrant inserts a newline after the User-Agent
header:

HEAD /wheezy64.box HTTP/1.1
User-Agent: Vagrant/1.6.5

Host: vagrant.1024.lu
Accept: application/json

HTTP/1.1 400 Bad Request
Date: Tue, 28 Oct 2014 08:03:46 GMT
Server: Apache
Vary: Accept-Encoding
Connection: close
Content-Type: text/html; charset=iso-8859-1

It comes from the VERSION string which contains a newline.
Adding a ".chomp" call (see attached patch) resolves the issue.

Note that this actually breaks every HTTP request so it makes vagrant
barely useable. It works only if you downloaded an image with a
previous version, or with file:// urls, so the severity should be
raised IMHO.

Cheers and thanks for taking care of Vagrant!

-- 
Etienne Millon
From c318c4152404ac4beaa78d14d55cd7e31dcaaae9 Mon Sep 17 00:00:00 2001
From: Etienne Millon 
Date: Tue, 28 Oct 2014 09:12:51 +0100
Subject: [PATCH] Chomp newline after version number

---
 debian/changelog   | 7 +++
 .../0003-VERSION-fallback-to-usr-share-vagrant-version.txt.patch   | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 85e5904..8ffcc98 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+vagrant (1.6.5+dfsg1-1.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Chomp newline after version number (Closes: #765074)
+
+ -- Etienne Millon   Tue, 28 Oct 2014 09:11:45 +0100
+
 vagrant (1.6.5+dfsg1-1) unstable; urgency=medium
 
   [ Antonio Terceiro ]
diff --git a/debian/patches/0003-VERSION-fallback-to-usr-share-vagrant-version.txt.patch b/debian/patches/0003-VERSION-fallback-to-usr-share-vagrant-version.txt.patch
index 9c2cd03..775f20f 100644
--- a/debian/patches/0003-VERSION-fallback-to-usr-share-vagrant-version.txt.patch
+++ b/debian/patches/0003-VERSION-fallback-to-usr-share-vagrant-version.txt.patch
@@ -21,7 +21,7 @@ index 0640365..22f6188 100644
 +VERSION = File.read(
 +  File.expand_path("../../../version.txt", __FILE__)).chomp
 +  rescue Errno::ENOENT
-+VERSION = File.read('/usr/share/vagrant/version.txt')
++VERSION = File.read('/usr/share/vagrant/version.txt').chomp
 +  end
  end
 -- 
-- 
2.1.1



signature.asc
Description: Digital signature


Bug#746103: python-notify2: FTBFS: Tests failures

2014-10-24 Thread Etienne Millon
tag 746103 + patch
thanks

Hi,

The following patch which fixes the build.
I'll prepare a team upload for this.

Cheers

-- 
Etienne Millon
Index: debian/rules
===
--- debian/rules	(revision 31238)
+++ debian/rules	(working copy)
@@ -13,7 +13,7 @@
 
 ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
 override_dh_auto_test:
-	PYTHONS="$(PYTHON2) $(PYTHON3)" xvfb-run -a debian/runtests.sh
+	PYTHONS="$(PYTHON2) $(PYTHON3)" dbus-run-session -- xvfb-run -a debian/runtests.sh
 endif
 
 build-python%:


Bug#760963: rejects xml feed based on http server mie type

2014-09-28 Thread Etienne Millon
tag 760963 + patch
forwarded 760963 https://github.com/wking/rss2email/pull/40
thanks

Here is the patch I applied and forwarded.

-- 
Etienne Millon
From: Etienne Millon 
Date: Sun, 28 Sep 2014 21:56:41 +0200
Subject: [PATCH] Log a warning when Content-Type is not correct

When a feed is served with a wrong Content-Type, log a warning and not an error.
This prevents a "processing error" message from being displayed at the default
log level, which is confusing for a innocuous "error" like this.

Forwarded: https://github.com/wking/rss2email/pull/40
Bug-Debian: https://bugs.debian.org/760963
Signed-off-by: Etienne Millon 
---
 rss2email/feed.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/rss2email/feed.py b/rss2email/feed.py
index 55979a3..ec5c4f6 100644
--- a/rss2email/feed.py
+++ b/rss2email/feed.py
@@ -423,6 +423,9 @@ class Feed (object):
 _LOG.warning(
 'incorrectly declared encoding: {}: {}'.format(exc, self))
 warned = True
+elif (parsed.bozo and isinstance(exc, _feedparser.NonXMLContentType)):
+_LOG.warning('non XML Content-Type: {}: {}'.format(exc, self))
+warned = True
 elif parsed.bozo or exc:
 if exc is None:
 exc = "can't process"
-- 
2.1.0



Bug#688551: python3-dateutil: Fresh release (2.1) is available for a while (since 2012-03-28)

2014-09-28 Thread Etienne Millon
* Guido Günther  [140928 19:41]:
> I've uploaded a python-dateutil that ships a python2 and python3
> version to experimental. Could you check if it works for you?
> 
> I'm cc'ing the python3-dateutil maintainers on this one since we
> would need to remove python3-dateutils from sid then.

Hi Guido,

With the package from experimental I can build the new version of
guessit with no problems.

Thanks for taking care of this!

-- 
Etienne Millon


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



Bug#760963: rejects xml feed based on http server mie type

2014-09-26 Thread Etienne Millon
* W. Trevor King  [140924 11:20]:
> On Tue, Sep 09, 2014 at 09:13:57AM -0700, W. Trevor King wrote:
> > On Tue, Sep 09, 2014 at 10:58:53AM -0400, Joey Hess wrote:
> > > Perhaps if it's not a fatal problem it should not display as an
> > > "error"?
> > 
> > I problably should have done this a while ago.  It's in master now [1].
> 
> Actually, I'm going to claw that back, since we don't want things like
> #760078 slipping by as warnings.  Maybe I should change 'processing
> error' to 'feedparser bozo'?

The code already does a big switch depending on exc's class.

IMHO it's best to add an extra case to display a warning if exc is a
NonXMLContentType exception (like the case just above for
CharacterEncodingOverride), and to keep the existing behaviour (log an
error) for other exception classes, which will catch stuff like
#760078.

I'll send you a patch if this sounds good to you.

Cheers

-- 
Etienne Millon


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



Bug#688551: python3-dateutil: Fresh release (2.1) is available for a while (since 2012-03-28)

2014-09-26 Thread Etienne Millon
* Tobias Hansen  [140926 15:20]:
> since version 2.2 can be used for Python 2 and 3 I think this affects
> also python-dateutil.

Hi,

I'm maintaining guessit that depends (in setup.py) on python-dateutil
(see #762957, in CC). 

The new version depends on python-dateutil>=2.1, but the python 2 & 3
versions are built by different source packages and have different
version numbers: py2 is <2 and py3 is >=2. So the build fails.

One possibility is to patch the src:guessit in order to remove the
bound, but since the newer versions of the py3 fork also work on
python 2, it seems better to merge the two packages. I don't think
it's too late for Jessie.

I believe that the correct course of action would be build the latest
2.x version from src:python-dateutil (including the python3-dateutil
binary package), then request removal of src:python3-dateutil.

Guido, if you don't have time to do that, the Python Modules Team
(including myself) can help with that. What do you think?

Cheers

-- 
Etienne Millon


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



Bug#762957: guessit: Please package version 0.9.3

2014-09-26 Thread Etienne Millon
Source: guessit
Severity: wishlist

Hello,

Version 0.9.3 is out, just a reminder that it should be packaged once
its dependencies are ready.

-- 
Etienne Millon


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



Bug#760969: blows away file permissions on rss2email.cfg

2014-09-26 Thread Etienne Millon
* W. Trevor King  [140924 11:20]:
> On Tue, Sep 09, 2014 at 11:06:26AM -0400, Joey Hess wrote:
> > Each time it is used to update list of feeds, r2e rewrites
> > rss2email.cfg, and neglects to preserve its file permissions when
> > doing so.
> 
> Hmm.  It's possible we can use shutil.copy2 to seed the new versions
> [1].  The approach would be:
> 
> * copy2 to create the new file, preserving as much metadata as
>   possible
> * truncate the new file
> * fill it in with the new data
> * move it over the original file
> 
> I'm also open to other approaches that still address atomic writes
> [2,3].

Hi,

I think that you can just add a call to shutil.copystat (that's what
copy2 uses) between creating the tempfile and the call to rename.

As an aside, I'm surprised that *every* Python application has to
handle all of this itself. There's probably room for a (small) library
here.

-- 
Etienne Millon


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



Bug#739030: severity of 739030 is serious, severity of 755207 is serious

2014-09-03 Thread Etienne Millon
Hello Michael,

I saw that you raised the severity of #739030 to a RC level for the
upcoming removal of valac-0.16.

I prepared an upload that makes it possible to build gmpc with a recent vala.
If you're interested in sponsoring this fix, I filed a RFS as #759535.

Thanks!

-- 
Etienne Millon


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



Bug#760078: rss2email: No longer able to access password-protected feeds

2014-09-02 Thread Etienne Millon
tag 760078 + patch
thanks

I'll go with the following patch. The rest of the code manipulates
decoded strings so it's necessary to redecode the result of
base64 encoding.

-- 
Etienne Millon
Subject: Fix HTTP auth in Python 3
From: Etienne Millon 
Bug-Debian: http://bugs.debian.org/760078

--- a/feedparser/feedparser.py
+++ b/feedparser/feedparser.py
@@ -2999,7 +2999,7 @@
 user_passwd, realhost = urllib.splituser(realhost)
 if user_passwd:
 url_file_stream_or_string = '%s://%s%s' % (urltype, realhost, rest)
-auth = base64.standard_b64encode(user_passwd).strip()
+auth = base64.standard_b64encode(user_passwd.encode('ascii')).strip().decode('ascii')
 
 # iri support
 if isinstance(url_file_stream_or_string, unicode):


Bug#760078: rss2email: No longer able to access password-protected feeds

2014-09-02 Thread Etienne Millon
reassign 760078 python3-feedparser
retitle 760078 HTTP auth does not work on python3
thanks

* Liam Morland  [140902 08:15]:
> 2014-09-01 17:40-0700 "W. Trevor King"  wrote: [...]
> >  auth = base64.standard_b64encode(user_passwd.encode('ASCII')).strip()
> >
> >worked for me.  If you're using Debian's python3-feedparser, you'll
> >want to file a bug with them [6] (although Etienne maintains both
> >python3-feedparser and rss2email for Debian, so you may not actually
> >need to file a new bug ;).
> 
> Thanks very much, Trevor.
> 
> Etienne, it is convienent that you maintain both packages. I'll leave it 
> to you.
> 
> Liam

Thanks to both of you. This is indeed a problem with feedparser, as
demonstrated by the attached test case. I'm not 100% satisfied with
using encoding ASCII as an encoding but unfortunately the standard
seems to mandate latin1 for non-ASCII characters...

http://stackoverflow.com/questions/702629/utf-8-characters-mangled-in-http-basic-auth-username

-- 
Etienne Millon
import datetime
import PyRSS2Gen

from flask import Flask, render_template, Response
from flask.ext.basicauth import BasicAuth

app = Flask(__name__)

app.config['BASIC_AUTH_USERNAME'] = 'user'
app.config['BASIC_AUTH_PASSWORD'] = 'pass'

basic_auth = BasicAuth(app)

@app.route('/rss.xml')
@basic_auth.required
def view_feed():
rss = PyRSS2Gen.RSS2(
title = "rss feed",
link = "http://localhost:5000/";,
description = "description",
lastBuildDate = datetime.datetime.now(),
items = [])
xml = rss.to_xml()
return Response(xml, mimetype='application/rss+xml')

if __name__ == '__main__':
app.run(debug=True)
import feedparser
URL = 'http://user:pass@localhost:5000/rss.xml'
p = feedparser.parse(URL)
print (p)


Bug#760078: rss2email: No longer able to access password-protected feeds

2014-09-01 Thread Etienne Millon
* Liam K Morland  [140901 08:48]:
> After a recent upgrade to rss2email, it is no longer able to access
> password-protected feeds. These used to be accessible via the
> following syntax:
> 
> r2e add http://user:passw...@example.com/feed
> 
> Upon "r2e run", it gives a confusing error message if it has any such 
> feeds in the database:
> 
> processing error: 'str' does not support the buffer interface

Hi,

In rss2email 3.x, "r2e add" takes two parameters, name and url.

Can you try the following command?

r2e add example http://user:passw...@example.com/feed

(whatever is the cause, I agree that the cryptic error message is a
bug in itself).

Thanks!

-- 
Etienne Millon


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



Bug#759978: [Help] Re: Bug#759978: seqan: FTBFS: c++: error: unrecognized command line option '-strong'

2014-09-01 Thread Etienne Millon
Hi Andreas,

debian/rules does the following thing:

export CXXFLAGS:=$(shell dpkg-buildflags --get CXXFLAGS | sed 
's/-fstack-protector *//')

This changes -fstack-protector-strong into -strong (which does not
exist).

You can do the same in a future-proof way with something like:

export DEB_BUILD_MAINT_OPTIONS = hardening=-stackprotector

HTH,

-- 
Etienne Millon


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



Bug#739030: gmpc: Upcoming vala-0.16 removal

2014-08-28 Thread Etienne Millon
unblock 739030 by 749055
thanks

* Andreas Henriksson  [140828 08:40]:
> Yesterday I briefly looked into what it would take to port gmpc to
> newer vala. At first I just had to change a function prototype where
> bindings had changed from ...(out x, out y, ...) to
> ...(ref x, ref y, ...).
> After that additional problems came up so I looked a bit at recent
> upstream VCS history Looks like they haven't switched to newer
> vala their either since they where still using the old ("out") function
> prototype.

Hello,

Indeed this works. I picked a commit from upstream which does exactly
this and fixes the build under valac 0.24.

I prepared a package and opened a RFS (#759535) if you (or someone in
your team) are interested in sponsoring this upload.

Thanks!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#759535: RFS: gmpc/11.8.16-9

2014-08-28 Thread Etienne Millon
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "gmpc"

 * Package name: gmpc
   Version : 11.8.16-9
   Upstream Author : QBall Cow
 * URL : http://gmpc.wikia.com/
 * License : GPL-2+
   Section : sound

It builds those binary packages:

 gmpc   - GNOME Music Player Client (graphical interface to MPD)
 gmpc-data  - GNOME Music Player Client - data files
 gmpc-dbg   - GNOME Music Player Client - debugging symbols
 gmpc-dev   - GNOME Music Player Client (plugin development files)

To access further information about this package, please visit the following 
URL:

http://mentors.debian.net/package/gmpc

Alternatively, one can download the package with dget using this command:

  dget -x http://mentors.debian.net/debian/pool/main/g/gmpc/gmpc_11.8.16-9.dsc

Changes since the last upload:

 gmpc (11.8.16-9) unstable; urgency=medium
 .
   * Fix building with a recent vala (Closes: #739030)
   * Improve package descriptions:
 - don't mention irrelevant details (Closes: #756303)
 - fix capitalization of GNOME
   * Bump Standards-Version to 3.9.5 (no changes).
   * Add libappindicator support (Closes: #736261)
   * Update Vcs-Browser URL.

Thanks!

-- 
Etienne Millon


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



Bug#555819: r2e download now all message smultipe times...

2014-07-31 Thread Etienne Millon
* Michelle Konzack  [140731 09:46]:
> Hello Maintainer,
> 
> I am now using rss2email since some month and it worked perfectly,
> even with a huge bunch of rss feeds   but  NOW  since  some  days
> it  starts downloading any messages again with each run.
> 
> It seems, I have hit a limit in r2e.
> 
> I have now installed a procmail recipe which filter by subject, but
> this is no long-term solution, because I get with this  bug  more
> then  4000 messages a day and have to cleanup this crap manualy.
> 
> However, I would suggest a rewrite of r2e which use  one  database
> file per RSS-Feed (I am sightly sure it  is  a  size  limit  because
> it  has 996834 bytes and does not more grow since several days)
> 
> I do not think, it is a good option to hack my own r2e script.
> 
> If you have an idea how to debug this error, please let me know.

Hello Michelle,

I believe that this behaviour has been fixed in the 3.x branch of
rss2email which changes how the database is stored and it should be
more robust. You can install a version from testing, migrate your data
using r2e-migrate and check if the problem still exists.

Thanks for testing it with lots of feeds :)

-- 
Etienne Millon


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



Bug#362483: rss2email: Unabled to disable 301 redirection for select feeds

2014-07-31 Thread Etienne Millon
* Keith Hellman  [140731 09:14]:
> I access a number of 'private' feeds associated with my university
> research group.  rss2email detects the 301 redirection and saves the
> new (redirected) url --- but this disables access next time an
> attempt is made.
> 
> I've attached a patch that uses a new command 'explicit' to prevent
> select feeds from respecting the embedded redirection information.

Hello,

I've been triaging old bugs and stumbled upon this one. The semantics
of a 301 redirection are clear, and according to RFC2616, rss2email as
a "clients with link editing capabilities" should change the URL in
its database. I don't know the specifics of these private URLs but
maybe another redirection such as 302 would be better.

Thanks for your bug report.

-- 
Etienne Millon


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



Bug#756303: gmpc: Improvement to package description

2014-07-29 Thread Etienne Millon
* Reuben Thomas  [140728 17:53]:
> > > A GTK2 client for Music Player Daemon. Features include:
> >
> > Please, s/GTK2/graphical/: this kind of technical term doesn’t have its
> > place inside the package description.
> >
> 
> Even better. I was about to note that in any case it will be a GTK3 app in
> its next release. Although, I think many users are interested in knowing
> whether an application is "native" to their favourite desktop (in
> particular, GNOME & KDE users).

Thanks for the suggestions, I will apply this change. As for the
"native" aspect, I think that users that want this are used to the
convention that g is a GTK application.

-- 
Etienne Millon


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



Bug#755206: gmpc: Upcoming vala-0.16 removal

2014-07-25 Thread Etienne Millon
forcemerge 739030 755206
thanks

* po...@debian.org  [140721 08:38]:
> gmpc build-depends on vala-0.16, which is pretty old and we would
> like to remove. You should instead use the much newer valac-0.24, or
> preferably the unversioned 'valac' package.

Hi,

I have started to work on this. I will first update libmpd and package
a newer gmpc. Thanks!

-- 
Etienne Millon


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



Bug#752458: rss2email: missing documentation for configuration file

2014-06-30 Thread Etienne Millon
tag 752458 + patch
forwarded 752458 https://github.com/wking/rss2email/pull/31
thanks

I submitted a patch upstream, attached here for completeness.

-- 
Etienne Millon
>From 56d9e7544f9058902a7413625a92b67a70accdd4 Mon Sep 17 00:00:00 2001
From: Etienne Millon 
Date: Mon, 30 Jun 2014 19:43:08 +0200
Subject: [PATCH] Document configuration items in manpage

This adds current items to the "CONFIGURATION" section in r2e.1, manually
extracted from rss2email/config.py.

Signed-off-by: Etienne Millon 
---
 r2e.1 | 134 ++
 1 file changed, 134 insertions(+)

diff --git a/r2e.1 b/r2e.1
index 4079399..df08752 100644
--- a/r2e.1
+++ b/r2e.1
@@ -134,6 +134,140 @@ use-publisher-email = True
 name-format = {author} ({feed.title})
 .RE
 .P
+You can configure the following items:
+.SS Addressing
+.RS
+.IP from
+The email address messages are from by default
+.IP use-8bit
+Transfer-Encoding. For local mailing it is safe and
+convient to use 8bit.
+.IP force-from
+True: Only use the 'from' address.
+False: Use the email address specified by the feed, when possible.
+.IP use-publisher-email
+True: Use the publisher's email if you can't find the author's.
+False: Just use the 'from' email instead.
+.IP name-format
+If empty, only use the feed email address rather than
+friendly name plus email address.  Available attributes may
+include 'feed', 'feed-title', 'author', and 'publisher', but
+only 'feed' is guaranteed.
+.IP to
+Set this to default To email addresses.
+.RE
+.SS Fetching
+.RS
+.IP proxy
+Set an HTTP proxy (e.g. 'http://your.proxy.here:8080/')
+.IP feed-timeout
+Set the timeout (in seconds) for feed server response
+.RE
+.SS Processing
+.RS
+.IP active
+True: Fetch, process, and email feeds.
+False: Don't fetch, process, or email feeds
+.IP digest
+True: Send a single, multi-entry email per feed per rss2email run.
+False: Send a single email per entry.
+.IP date-header
+True: Generate Date header based on item's date, when possible.
+False: Generate Date header based on time sent.
+.IP date-header-order
+A comma-delimited list of some combination of
+('issued', 'created', 'modified', 'expired')
+expressing ordered list of preference in dates
+to use for the Date header of the email.
+.IP bonus-header
+Set this to add bonus headers to all emails
+Example: bonus-header = 'Approved: j...@bob.org'
+.IP trust-guid
+True: Receive one email per post.
+False: Receive an email every time a post changes.
+.IP trust-link
+True: Receive one email per unique link url.
+False: Defer to trust-guid preference.
+Toggling this for existing feeds may result in duplicates,
+because the old entries will not be recorded under their new
+link-based ids.
+.IP encodings
+To most correctly encode emails with international
+characters, we iterate through the list below and use the
+first character set that works.
+.IP post-process
+User processing hooks.  Note the space after the module name.
+Example: post-process = 'rss2email.post_process.downcase downcase_message'
+.IP digest-post-process
+User processing hooks for digest messages.  If 'digest' is
+enabled, the usual 'post-process' hook gets to massage the
+per-entry messages, but this hook is called with the full
+digest message before it is mailed.
+Example: digest-post-process = 'rss2email.post_process.downcase downcase_message'
+.RE
+.SS HTML conversion
+.RS
+.IP html-mail
+True: Send text/html messages when possible.
+False: Convert HTML to plain text.
+.IP use-css
+Use CSS
+.IP css
+Optional CSS styling
+.RE
+.SS html2text options
+.RS
+.IP unicode-snob
+Use Unicode characters instead of their ascii psuedo-replacements
+.IP links-after-each-paragraph
+Put the links after each paragraph instead of at the end.
+.IP body-width
+Wrap long lines at position. 0 for no wrapping.
+.RE
+.SS Mailing
+.RS
+.IP email-protocol
+Select protocol from: sendmail, smtp, imap
+.IP sendmail
+Path to sendmail (or compatible)
+.RE
+.SS SMTP configuration
+.RS
+.IP smtp-auth
+Set to True to use SMTP AUTH
+.IP smtp-username
+username for SMTP AUTH
+.IP smtp-password
+password for SMTP AUTH
+.IP smtp-server
+SMTP server
+.IP smtp-ssl
+Connect to the SMTP server using SSL
+.RE
+.SS IMAP configuration
+.RS
+.IP imap-auth
+set to True to use IMAP auth.
+
+.IP imap-username
+username for IMAP authentication
+.IP imap-password
+password for IMAP authentication
+.IP imap-server
+IMAP server
+.IP imap-port
+IMAP port
+.IP imap-ssl
+connect to the IMAP server using SSL
+.IP imap-mailbox
+where we should store new messages
+.RE
+.SS Miscellaneous
+.RS
+.IP verbose
+Verbosity (one of 'error', 'warning', 'info', or 'debug').
+.RE
+.P
 .SH FILES
 .TP 4
 .B $XDG_CONFIG_HOME/rss2email.cfg
-- 
2.0.0



Bug#752522: rss2email: does nothing after today's upgrade

2014-06-29 Thread Etienne Millon
tag 752522 + patch
thanks

Here is a patch that displays a message if no feeds are loaded and a
2.x DB exists.

-- 
Etienne Millon
From: Etienne Millon 
Forwarded: not-needed
Bug-Debian: https://bugs.debian.org/752522
Subject: Detect 2.x database

As noted in #752522, it is quite confusing to have "r2e list" display nothing
after an upgrade from 2.x to 3.x. Even though it is in NEWS.Debian, not all
users read it so it is better to display a message if a 2.x database exists and
no feeds are found.

This is specific to the wheezy -> jessie upgrade, so will probably be dropped
after jessie is released.

--- a/rss2email/main.py
+++ b/rss2email/main.py
@@ -19,6 +19,7 @@
 
 import argparse as _argparse
 import logging as _logging
+import os as _os
 import sys as _sys
 
 from . import __doc__ as _PACKAGE_DOCSTRING
@@ -37,6 +38,11 @@
 _sys.exit(0)
 
 
+def v2_database_exists():
+path = _os.path.expanduser('~/.rss2email/feeds.dat')
+return _os.path.isfile(path)
+
+
 def run(*args, **kwargs):
 """The rss2email command line interface
 
@@ -160,6 +166,12 @@
 if args.func != _command.new:
 lock = args.func not in [_command.list, _command.opmlexport]
 feeds.load(lock=lock)
+if v2_database_exists() and not feeds:
+print("A rss2email 2.x database exists, but it can't be used "
+  "directly by this version of rss2email (3.x).\n"
+  "You can migrate your data using r2e-migrate.\n"
+  "Please see /usr/share/doc/rss2email/NEWS.Debian.gz\n"
+  "and r2e-migrate(1) for more information.")
 args.func(feeds=feeds, args=args)
 except _error.RSS2EmailError as e:
 e.log()


Bug#752522: rss2email: does nothing after today's upgrade

2014-06-24 Thread Etienne Millon
retitle 752522 please detect if a 2.x database exist
severity 752522 minor
thanks

Hello,

rss2email 3.x changed its config file format, and now reads from a
different file with a different syntax. The r2e-migrate tool can help
you transfer your data.

However, an improvement is possible: it is possible to patch r2e to
check if a 2.x database exists and write a message about it. I'm
retitling the bug and setting the severity accordingly.

This breaking change is described in
/usr/share/doc/rss2email/NEWS.Debian.gz ; you can install the
apt-listchanges package which will warn you when you upgrade a package
that contains such an incompatible change.

Feel free to file a new bug if the migration does not work.

Sorry for the inconvenience, and thanks for your bug report!

-- 
Etienne Millon


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



Bug#729781: ITP: subliminal -- Command-line tool to search and download subtitles

2014-06-16 Thread Etienne Millon
* Felipe Sateler  [140614 23:45]:
> It seems dogpile is not available for python3.

Indeed! Filed as #751739, and it's currently sitting in NEW.

> Also, I would argue that the command line binary should go in a
> package not named python-*.

You're right. I moved it to a new binary package.

Thanks!

-- 
Etienne Millon


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



Bug#751739: python-dogpile.cache: Please provide a python3 package

2014-06-16 Thread Etienne Millon
Package: python-dogpile.cache
Version: 0.5.3-1
Severity: wishlist

Hi,

I'm packaging subliminal (ITP - #729781) and I would like to provide a
python3 package and use it for the CLI tool.

Can you please provide a python3-dogpile.cache package once #751662
(python3-dogpile.core) is fixed?

Thanks a lot!

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.14-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages python-dogpile.cache depends on:
ii  dpkg 1.17.10
ii  python   2.7.6-2
ii  python-dogpile.core  0.4.1+dfsg1-1

python-dogpile.cache recommends no packages.

python-dogpile.cache suggests no packages.

-- no debconf information

-- 
Etienne Millon


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



Bug#729781: ITP: subliminal -- Command-line tool to search and download subtitles

2014-06-13 Thread Etienne Millon
unblock 729781 by 698258
thanks

Now that the dependencies are in unstable (charade not needed so
unblocked), here is the package (WIP) on Alioth:

http://anonscm.debian.org/viewvc/python-modules/packages/subliminal/trunk/

-- 
Etienne Millon


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



Bug#749055: libmpd: Please package version 11.8.90

2014-05-23 Thread Etienne Millon
Source: libmpd
Severity: wishlist

Hello,

I maintain gmpc, from the same upstream author.
The version in testing builds only with vala-0.16 which is scheduled
for removal. This would leave gmpc RC-buggy (see #739030).

As a solution I would like to introduce a newer version, probably a
git snapshot. The problem is that it needs a recent libmpd version. No
release has been officially made but everything works with the latest
upstream commit from 2013-03-19.

I see that you're in the Low NMU threshold, if you want I can do the
work as part of a NMU, but as it is quite disruptive I prefer to ask
you first.

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.14-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

-- 
Etienne Millon


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



Bug#720941: zsnes: FTBFS on hurd-i386

2014-05-15 Thread Etienne Millon
* Svante Signell  [140515 22:57]:
> Sorry, there is a typo in the patch: change
> +#ifdef __GNU__
> +#include 
> +#else
> to
> +#ifdef __GNU__
> +#include 
> +#else
> then it builds.
> 
> HTH!

Thanks a lot, I will update the patch and reupload ASAP.

-- 
Etienne Millon


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



Bug#720941: zsnes: FTBFS on hurd-i386

2014-05-15 Thread Etienne Millon
reopen 720941
thanks

Hi Svante (and hurd porters),

I applied the fix that you provided in bug #720941 but it does not
seem to be enough. Please see the following build log:

https://buildd.debian.org/status/fetch.php?pkg=zsnes&arch=hurd-i386&ver=1.510%2Bbz2-7&stamp=1400076464

The relevant line is:

> make[1]: *** No rule to make target 'sys/ioctls.h', needed by 'initc.o'.  
> Stop.

How do you think this can be fixed?

Thanks!

-- 
Etienne Millon


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



Bug#740860: zsnes is randomly frozen

2014-04-27 Thread Etienne Millon
The patch looks good to me. Without the cast, the computation is done
on long arithmetic and the result is extended to unsigned long long.

I'll test it and release a new package if it's fine.

Thanks a lot for the debugging and the patch!

-- 
Etienne Millon


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



Bug#745488: python-dogpile.cache: please package version 0.5.3

2014-04-22 Thread Etienne Millon
Package: python-dogpile.cache
Version: 0.5.0-1
Severity: wishlist

Hello,

I am in the process of packaging subliminal (ITP - #729781), which
requires at least version 0.5.2 (pypi version is 0.5.3 ATM).

Can you please consider updating it?

Thanks!

-- 
Etienne Millon


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



Bug#742686: python3-babelfish: Infinite loop when getattr

2014-03-26 Thread Etienne Millon
Package: python3-babelfish
Version: 0.5.1-1
Severity: normal
Tags: patch upstream

This bug affects python3-babelfish for python3 >= 3.4.

https://github.com/Diaoul/babelfish/issues/16

Attached upstream commit fixes the issue.

Thanks

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.13-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages python3-babelfish depends on:
ii  python3-pkg-resources  2.2-1
pn  python3:any

python3-babelfish recommends no packages.

python3-babelfish suggests no packages.

-- no debconf information

-- 
Etienne Millon
>From 280b9cfe55f24ca5a26b5786a21fd4c0589f4139 Mon Sep 17 00:00:00 2001
From: Toilal 
Date: Wed, 26 Mar 2014 11:36:16 +0100
Subject: [PATCH] Fix bug #16.

---
 babelfish/country.py  | 2 +-
 babelfish/language.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/babelfish/country.py b/babelfish/country.py
index 83641fc..6cdd0fa 100644
--- a/babelfish/country.py
+++ b/babelfish/country.py
@@ -43,7 +43,7 @@ class CountryMeta(type):
 def __getattr__(cls, name):
 if name.startswith('from'):
 return partial(cls.fromcode, converter=name[4:])
-return getattr(cls, name)
+return type.__getattribute__(cls, name)
 
 
 class Country(CountryMeta(str('CountryBase'), (object,), {})):
diff --git a/babelfish/language.py b/babelfish/language.py
index ffa48ef..e2e6235 100644
--- a/babelfish/language.py
+++ b/babelfish/language.py
@@ -52,7 +52,7 @@ class LanguageMeta(type):
 def __getattr__(cls, name):
 if name.startswith('from'):
 return partial(cls.fromcode, converter=name[4:])
-return getattr(cls, name)
+return type.__getattribute__(cls, name)
 
 
 class Language(LanguageMeta(str('LanguageBase'), (object,), {})):
-- 
1.8.5.5



Bug#742215: rss2email: invalid User-Agent

2014-03-21 Thread Etienne Millon
* W. Trevor King  [140321 09:01]:
> On Thu, Mar 20, 2014 at 09:00:20PM +0100, Jakub Wilk wrote:
> > * W. Trevor King, 2014-03-20, 12:49:
> > >We should be able to use:
> > >
> > >  User-Agent: rss2email/3.8 (https://github.com/wking/rss2email)
> > >
> > >Does that sound legal to you?
> > 
> > Yes, looks go to me. :-)
> 
> Fixed in master's ec438df (feed: Change _USER_AGENT from '+{url}' to
> '({url})', 2014-03-20).

Hi,

Thanks for the report and the patch. I applied it in git in the
meantime.

-- 
Etienne Millon


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



Bug#740860: zsnes is randomly frozen

2014-03-05 Thread Etienne Millon
Hi,

I suspect that this is the same as LP#1269676:

https://bugs.launchpad.net/ubuntu/+source/zsnes/+bug/1269676

I could reproduce this bug (with another game) and managed to get a
core dump, but did not investigate deeper.

The two causes I suspect are either a change in binary compatibility
due to hardening (see #727781), or a multiarch issue with SDL.

Thanks for this bug report!

-- 
Etienne Millon


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



Bug#740385: rss2email: can't set verbosity in rss2email.cfg

2014-03-05 Thread Etienne Millon
tags 740385 + upstream
forwarded 740385 https://github.com/wking/rss2email/issues/28
stop

Thanks for the bug report, forwarding it in the upstream BTS (even
though upstream is here too :)).

-- 
Etienne Millon


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



Bug#740384: rss2email: fails to raise NoToEmailAddress: TypeError: __init__() missing 1 required positional argument: 'feed'

2014-02-28 Thread Etienne Millon
* W. Trevor King  [140228 23:07]:
> Oops, thanks.  Should be fixed in:
> 
>   git://github.com/wking/rss2email.git no-to-email-error
> 
> with 763d9a8 (command: Set the 'feed' argument when raising
> NoToEmailAddress, 2014-02-28).

I cherry-picked it, thanks for the quick fix!

-- 
Etienne Millon


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



Bug#740371: r2e-migrate: doesn't honour XDG_DATA_HOME: No such file or directory: '/home/jwilk/.local/share/rss2email.json'

2014-02-28 Thread Etienne Millon
* Jakub Wilk  [140228 20:50]:
> IOError: [Errno 2] No such file or directory: 
> '/home/jwilk/.local/share/rss2email.json'
> 
> I don't have ~/.local/, because I have XDG_DATA_HOME set to
> something else, and apparently r2e-migrate doesn't honour this
> variable.

Thanks again for testing this version.

Replacing ".local/share" by xdg.BaseDirectory.xdg_data_home should do
the trick. If the directory set by $XDG_DATA_HOME does not exist,
there will be an error message but I think that's the expected
outcome in that case.

-- 
Etienne Millon


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



Bug#740370: rss2email: “but not not the of” in NEWS.Debian

2014-02-28 Thread Etienne Millon
* Jakub Wilk  [140228 20:50]:
> NEWS.Debian entry for 1:3.5-1~exp1 reads: “[…], but not not the of
> already seen articles.” This doesn't look like correct English to
> me.

Thanks for this bug report. It looks like bad copy/pasta to me. I'll
change it to "but not the already-seen database".

-- 
Etienne Millon


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



Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-24 Thread Etienne Millon
* Alexandre Detiste  [140220 21:55]:
> Here is a new patch using new function "gdp_unzip_paths()", this
> should work for everybody. I tested it with all three avaible
> unzippers.

Hi,

I tested the patch and it works great. Once I installed the package
that it produced, I could run the game using opentyrian.

Thanks!

-- 
Etienne Millon


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



Bug#739486: game-data-packager: Please add support for Tyrian data using following patch.

2014-02-19 Thread Etienne Millon
Hi,

After applying this patch I to run "game-data-packager tyrian -w" and
got the following error:

> 2014-02-19 17:26:04 (1.10 MB/s) -
> '/tmp/game-data-packager.f61l7L/tyrian21.zip' saved
> [4754048/4754048]
> 
> rm: cannot remove '*.exe': No such file or directory rmdir: failed
> to remove '/tmp/game-data-packager.f61l7L': Directory not empty

I presume that's because you're in the wrong directory at:

> +gdp_unzip "$ZIPFILE"
> +rm *.exe
> +rm *.ovl
> +    rmdir tyrian21

(the rmdir is suspicious, too)

Thanks for your work!

-- 
Etienne Millon


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



Bug#739030: gmpc: Upcoming vala-0.16 removal

2014-02-16 Thread Etienne Millon
* Andreas Henriksson  [140215 23:26]:
> With latest upstream snapshot you'll also need the latest upstream
> snapshot of libmpd, which has bumped SONAME (not ABI compatible) so
> it'll need a transition in debian... 
> So this path is not very straight forward either unfortunately. :(

It seems that the SONAME is still 1, which will definitely make things
easier (gmpc uses the new priority field, though):

http://upstream-tracker.org/versions/libmpd.html

I'll contact Arnaud about this.

> I understand it will take some work and require some time to get
> gmpc ported to newer vala. I just wanted to make sure it's atleast
> on your todo list and maybe with a bit of a priority because if/when
> gmpc is the last thing that requires vala 0.16 we'll have to discuss
> how to deal with the situation

I wasn't aware of the situation, but upgrading it was part of my
plans. The priority has bumped a little :)

-- 
Etienne Millon


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



Bug#739030: gmpc: Upcoming vala-0.16 removal

2014-02-15 Thread Etienne Millon
Hi,

Well, half a year is not so bad :-/ Sorry for hindering the removal.

If I remember correctly, porting the current version to a recent vala
compiler would require a non negligible amount of work (by the way, do
you know if there is a porting guide for valac upgrades?). Upstream is
working on a new version but has not released one yet.

The best course of action is probably to package a git snapshot and
convince upstream to release before the freeze, as I'd like to provide
a newer gmpc for jessie (including glyr support etc).

Thanks

-- 
Etienne Millon


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



Bug#698258: ITP: python-charade -- universal encoding detector for Python 2 and Python 3

2014-02-14 Thread Etienne Millon
Hi,

According to pypi, charade has been merged into chardet.
Is packaging charade still relevant?

* https://pypi.python.org/pypi/chardet:
> This is a continuation of Mark Pilgrim's excellent chardet.
> Previously, two versions needed to be maintained: one that supported
> python 2.x and one that supported python 3.x. We've recently merged
> with Ian Corduscano's charade fork, so now we have one coherent
> version that works for Python 2.6+.

-- 
Etienne Millon


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



Bug#553035: willing to help

2014-02-13 Thread Etienne Millon
* Alexandre Detiste  [140129 09:24]:
> I use this game and I'm willing to help.

Hi,

Unfortunately not a lot has changed since my last message. Due to the
non-redistributability of opentyrian-data, it's impossible to create a
source package for it. For the moment I created an open-tyrian binary
package that installs the data files to
/usr/share/games/opentyrian/data. The zip can be found at the
following URL:

http://www.camanis.net/tyrian/tyrian21.zip

However it's not possible as is. This zip file should be downloaded
and unpacked during postinst to somewhere in /var (cache or lib). It
is also possible to do this before launching the game somewhere in
$HOME. The latter is probably better as it avoids to download as root.

In the meantime, if you want to test, you can remove the dependency
and launch the game in a directory where data/ contains game data (see
src/file.c).

For the actual download code, I don't know if it's possible to use
game-data-packager, but it's always possible to roll our own solution,
à la flashplugin-nonfree.

In the meantime, I refreshed my packaging, fixed a few 
warnings and pushed my package to the following URL:

git+ssh://git.debian.org/git/pkg-games/opentyrian.git

As you can see your help is welcome :)

You can add yourself to Uploaders and help fix the remaining lintian
warnings, do the copyright review (should be OK), and/or of course
find a solution for the data files :)

If it's your first package I'll be happy to help you.

Please let me know what you're interested in.

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#734327: rss2email "TypeError: sequence item 1: expected string or Unicode, NoneType found"

2014-02-13 Thread Etienne Millon
Hi Jeroen,

Sorry for the delay. As Trevor said, the development is more active on
the 3.x branch (still in experimental). I encourage you to try it
independently of this problem!

That being said, it looks more like a feedparser bug, though it can
also be fixed in rss2email (the tag handling code is the same in 3.x).

To be sure, can you attach a copy of the feed when it fails? I'm sure
it will help other programs that use feedparser.

Thanks!

-- 
Etienne Millon


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



Bug#738102: rss2email: "The feed has not changed since you last checked"

2014-02-08 Thread Etienne Millon
reassign 738102 feedparser
retitle 738102 wrong status code after redirection
tag 738102 + patch
thanks

Hi,

It seems to be a feedparser bug, reassigning accordingly. When a 302
redirects to a 304, the caller just sees the 302.

This bug is tracked upstream at

https://code.google.com/p/feedparser/issues/detail?id=390

Until a solution is accepted upstream I'll patch feedparser to return
the 304 to the caller.

Thanks!

-- 
Etienne Millon
Subject: Fix status reported for redirections
From: jikam...@gmail.com
Bug: https://code.google.com/p/feedparser/issues/detail?id=390
Bug-Debian: http://bugs.debian.org/738102
Reviewed-by: Etienne Millon 

--- a/feedparser/feedparser.py
+++ b/feedparser/feedparser.py
@@ -3971,6 +3971,7 @@
 
 # Stop processing if the server sent HTTP 304 Not Modified.
 if getattr(f, 'code', 0) == 304:
+result['status'] = 304
 result['version'] = u''
 result['debug_message'] = 'The feed has not changed since you last checked, ' + \
 'so the server sent no data.  This is a feature, not a bug!'
--- a/feedparser/feedparsertest.py
+++ b/feedparser/feedparsertest.py
@@ -523,7 +523,7 @@
 u = 'http://localhost:8097/tests/http/http_redirect_to_304.xml'
 f = feedparser.parse(u)
 self.assertTrue(f.bozo == 0)
-self.assertTrue(f.status == 302)
+self.assertTrue(f.status == 304)
 
 class TestDateParsers(unittest.TestCase):
 "Test the various date parsers; most of the test cases are constructed " \


signature.asc
Description: Digital signature


Bug#737909: stevedore: Please package version 0.14.1

2014-02-06 Thread Etienne Millon
Source: stevedore
Severity: wishlist

Hi,

I'm packaging the guessit library (ITP - #729870), which uses the
newer features of stevedore (namely the "on_load_failure_callback"
parameter to ExtensionManager.__init__).

Can you consider packaging the newest version?

Thanks!

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.12-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

-- 
Etienne Millon


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



Bug#737029: python-babelfish: Missing dependency on python-pkg-resources

2014-02-03 Thread Etienne Millon
* Martin Pitt  [140203 16:13]:
> Indeed, we see the same in Ubuntu:
> 
>   
> https://jenkins.qa.ubuntu.com/job/trusty-adt-babelfish/4/ARCH=i386,label=adt/console
> 
> This was previously hidden because the -nose test dependency pulls in
> pkg_resources. Trivial debdiff attached for reference, tested with
> 
>   adt-run --unbuilt-tree=. --- adt-virt-schroot sid

Thanks for the patch. I prepared 0.5.1-1 with a similar patch, but I
preferred to wait for 0.4.0-2 to reach testing before uploading it.
I'll do it ASAP.

-- 
Etienne Millon


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



Bug#737398: PTS is confused by trailing commas in Uploaders

2014-02-02 Thread Etienne Millon
Package: qa.debian.org
Severity: minor
Tags: patch

Hi,

It seems that the PTS is confused by fields containing trailing
commas (as inserted by wrap-and-ort -t).

For example on the PTS entry of babelfish, it displays

"maint Debian Python Modules Team (a), Etienne Millon (u), Oxan van
Leeuwen (u), (u)"

http://packages.qa.debian.org/b/babelfish.html

The attached patch seems to fix the issue by filtering empty strings,
but I'm not too familiar with the PTS source code so testing is
advised.

Thanks!

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.12-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

-- 
Etienne Millon
Index: www/bin/sources_to_xml.py
===
--- www/bin/sources_to_xml.py	(revision 3108)
+++ www/bin/sources_to_xml.py	(working copy)
@@ -29,7 +29,8 @@
 # Just as address_from_string, it tries to be forgiving about unquoted
 # commas in addresses. [PvR]
 def addresses_from_string(content):
-return map(address_from_string, re.split('(?<=>)\s*,\s*', content))
+matches = re.split('(?<=>)\s*,\s*', content)
+return [address_from_string(m) for m in matches if m]
 
 def add_maintainer_info(child, name, mail, doc):
 text = doc.createTextNode(unicode(name, 'UTF-8', 'replace')) # Take care of non-ascii


Bug#737029: python-babelfish: Missing dependency on python-pkg-resources

2014-01-29 Thread Etienne Millon
Package: python-babelfish
Version: 0.4.0-2
Severity: normal

Hi,

Python-babelfish is using the pkg_resources module, but does not
depend on the package providing it. It makes autopkgtests fail:

http://ci.debian.net/#package/babelfish

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (900, 'testing')
Architecture: i386 (i686)

Kernel: Linux 3.12-1-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages python-babelfish depends on:
pn  python:any  

python-babelfish recommends no packages.

python-babelfish suggests no packages.

-- no debconf information

-- 
Etienne Millon


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



Bug#733953: babelfish: Autopkgtest failure because of stderr output

2014-01-27 Thread Etienne Millon
tag 733953 - moreinfo
tag 733953 + pending
thanks

Great, I updated devscripts and it worked.
I'll upload the package ASAP.

Thanks again!

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#733953: babelfish: Autopkgtest failure because of stderr output

2014-01-27 Thread Etienne Millon
tag 733953 + moreinfo
thanks

Hi,

Thanks for the bug report and the patch.
I also saw the package fail tests on ci.d.n:

http://ci.debian.net/#package/babelfish

I'm still new to DEP8 and can't reproduce the bug.
I tried to create a sid chroot and invoke adt-run with the following
line, but it succeeds.

adt-run babelfish_0.4.0-1.dsc \
--- adt-virt-chroot \
-r sudo \
/path/to/sid/chroot 

Do you know how can I see the failure ?

> - Please run tests with --verbose (-v) option;
> - Nose is deprecated, please consider using either pure unittest or nose2.

I'll drop the test-dependencies on nose and invoke the suite using
"python -m unittest -v babelfish.tests" (+py3 alternative).

Thanks

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#731542: libglyr1: basically useless lyrics search

2013-12-06 Thread Etienne Millon
tags 731542 + patch
thanks

Hi,

The problem is that the lyricswiki provider is always succesful. It
has been fixed in the following commit:

https://github.com/sahib/glyr/commit/effb0b0099e94f83bff7458149adbffe74083561

Adding this as a patch fixes the problem, but you're right that it
would be nice to cut a release as there are a lot of pending fixes not
yet out. I'll contact upstream to see what he thinks.

Cheers

-- 
Etienne Millon


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



Bug#729870: license file

2013-11-29 Thread Etienne Millon
The library is LGPL3 but does not ship the full text of the license, I
contacted upstream about this issue:

https://github.com/wackou/guessit/issues/69

-- 
Etienne Millon


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



Bug#724200: muttprint: FTBFS: POD errors

2013-11-29 Thread Etienne Millon
tags 724200 + patch
thanks

Hi,

Here is a fix for this bug.
I'll look for a sponsor for NMU in a few days unless you object.

Thanks!

-- 
Etienne Millon
diff -u muttprint-0.73/debian/changelog muttprint-0.73/debian/changelog
--- muttprint-0.73/debian/changelog
+++ muttprint-0.73/debian/changelog
@@ -1,3 +1,11 @@
+muttprint (0.73-4.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fix "FTBFS: POD errors"
+Add a patch with encoding (Closes: #724200)
+
+ -- Etienne Millon   Fri, 29 Nov 2013 11:55:24 +0100
+
 muttprint (0.73-4) unstable; urgency=low
 
   * remove obsolete debconf note (closes: #631303, #658505)
diff -u muttprint-0.73/debian/patches/00list 
muttprint-0.73/debian/patches/00list
--- muttprint-0.73/debian/patches/00list
+++ muttprint-0.73/debian/patches/00list
@@ -8,0 +9 @@
+19_pod_encoding
only in patch2:
unchanged:
--- muttprint-0.73.orig/debian/patches/19_pod_encoding.dpatch
+++ muttprint-0.73/debian/patches/19_pod_encoding.dpatch
@@ -0,0 +1,39 @@
+#! /bin/sh /usr/share/dpatch/dpatch-run
+## 19_pod_encoding.dpatch by Etienne Millon 
+##
+## All lines beginning with `## DP:' are a description of the patch.
+## DP: Add encoding to fix FTBFS (#724200)
+
+@DPATCH@
+diff -urNad '--exclude=CVS' '--exclude=.svn' '--exclude=.git' 
'--exclude=.arch' '--exclude=.hg' '--exclude=_darcs' '--exclude=.bzr' 
muttprint~/doc/manpages/de/muttprint.pod muttprint/doc/manpages/de/muttprint.pod
+--- muttprint~/doc/manpages/de/muttprint.pod   2013-11-29 12:07:05.0 
+0100
 muttprint/doc/manpages/de/muttprint.pod2013-11-29 12:07:05.0 
+0100
+@@ -1,3 +1,4 @@
++=encoding latin1
+ =head1 NAME
+ 
+ muttprint - sch�ner Ausdruck von Mails mit Mutt
+diff -urNad '--exclude=CVS' '--exclude=.svn' '--exclude=.git' 
'--exclude=.arch' '--exclude=.hg' '--exclude=_darcs' '--exclude=.bzr' 
muttprint~/doc/manpages/en/muttprint.pod muttprint/doc/manpages/en/muttprint.pod
+--- muttprint~/doc/manpages/en/muttprint.pod   2013-11-29 12:07:05.0 
+0100
 muttprint/doc/manpages/en/muttprint.pod2013-11-29 12:07:28.524841284 
+0100
+@@ -1,3 +1,4 @@
++=encoding latin1
+ =head1 NAME
+ 
+ muttprint - pretty printing of mails with Mutt
+diff -urNad '--exclude=CVS' '--exclude=.svn' '--exclude=.git' 
'--exclude=.arch' '--exclude=.hg' '--exclude=_darcs' '--exclude=.bzr' 
muttprint~/doc/manpages/es/muttprint.pod muttprint/doc/manpages/es/muttprint.pod
+--- muttprint~/doc/manpages/es/muttprint.pod   2013-11-29 12:07:05.0 
+0100
 muttprint/doc/manpages/es/muttprint.pod2013-11-29 12:07:46.212928995 
+0100
+@@ -1,3 +1,4 @@
++=encoding latin1
+ =head1 NOMBRE
+ 
+ muttprint - impresi�n elegante de mensajes de correo con Mutt
+diff -urNad '--exclude=CVS' '--exclude=.svn' '--exclude=.git' 
'--exclude=.arch' '--exclude=.hg' '--exclude=_darcs' '--exclude=.bzr' 
muttprint~/doc/manpages/it/muttprint.pod muttprint/doc/manpages/it/muttprint.pod
+--- muttprint~/doc/manpages/it/muttprint.pod   2013-11-29 12:07:05.0 
+0100
 muttprint/doc/manpages/it/muttprint.pod2013-11-29 12:07:40.584901087 
+0100
+@@ -1,3 +1,4 @@
++=encoding latin1
+ =head1 NOME
+ 
+ muttprint - stampa raffinata della posta elettronica con Mutt


signature.asc
Description: Digital signature


Bug#729872: RFS: DPMT: pysrt 0.5.1-1

2013-11-27 Thread Etienne Millon
(cc-ing the ITP bug)

* Piotr Ożarowski  [131127 01:51]:
> I just noticed that pysrt is GPL but doesn't include full version of
> the licence in the tarball, please notify upstream that he needs to
> include it.

Upstream is aware of this problem, and I posted a message on the
upstream bug tracker there:

https://github.com/byroot/pysrt/issues/34

I'll ping you when a new version released with the correct license
file.

-- 
Etienne Millon


signature.asc
Description: Digital signature


Bug#729870: ITP: guessit -- Python library for guessing information from video filenames

2013-11-24 Thread Etienne Millon
Pushed to alioth:

Vcs-Svn: svn://anonscm.debian.org/python-modules/packages/guessit/trunk/
Vcs-Browser: 
http://anonscm.debian.org/viewvc/python-modules/packages/guessit/trunk/

-- 
Etienne Millon


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



Bug#729868: ITP: enzyme -- Python module to parse video metadata

2013-11-24 Thread Etienne Millon
Pushed on alioth:

Vcs-Svn: svn://anonscm.debian.org/python-modules/packages/enzyme/trunk/
Vcs-Browser: 
http://anonscm.debian.org/viewvc/python-modules/packages/enzyme/trunk/

-- 
Etienne Millon


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



Bug#729869: ITP: babelfish -- Python library to work with countries and languages

2013-11-24 Thread Etienne Millon
Pushed to alioth:

Vcs-Svn: svn://anonscm.debian.org/python-modules/packages/babelfish/trunk/
Vcs-Browser: 
http://anonscm.debian.org/viewvc/python-modules/packages/babelfish/trunk/

-- 
Etienne Millon


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



Bug#729872: ITP: pysrt -- Python parser for SubRip (srt) files

2013-11-24 Thread Etienne Millon
Pushed to Alioth:

Vcs-Svn: svn://anonscm.debian.org/python-modules/packages/pysrt/trunk/
Vcs-Browser: 
http://anonscm.debian.org/viewvc/python-modules/packages/pysrt/trunk/

-- 
Etienne Millon


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



Bug#729781: ITP: subliminal -- Command-line tool to search and download subtitles

2013-11-24 Thread Etienne Millon
* Daniele Tricoli  [131124 11:40]:
> I'm also a DPMT member and I sent a RFS for charade months ago.
> Something did go wrong and it was not uploaded:
> http://anonscm.debian.org/viewvc/python-modules/packages/python-charade/
> 
> It was tagged but I don't know why it did not get uploaded. I will
> work on this now: I will also update it to use pybuild.
> 
> Etienne, I can work on charade, since I'm also interested on having
> it in Debian, so you can focus on the other packages, if it's not a
> problem for you :)

Sure, I wasn't planning on packaging charade as I saw your ITP.

Pysrt depends on charade, but I added a small patch to make it use
chardet in the meantime. So, it's not blocking for me.

Good luck for your RFS :)

-- 
Etienne Millon


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



Bug#729969: zsnes: add a debug package

2013-11-19 Thread Etienne Millon
Package: zsnes
Version: 1.510+bz2-6
Severity: minor

Given the recent stability problems (ex: LP:1214241, #727781), a
package with debugging symbols would be a nice addition to zsnes.

Thanks

-- 
Etienne Millon


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



Bug#729872: ITP: pysrt -- Python parser for SubRip (srt) files

2013-11-18 Thread Etienne Millon
Package: wnpp
Severity: wishlist
Owner: Etienne Millon 
X-Debbugs-Cc: python-modules-t...@lists.alioth.debian.org

* Package name: pysrt
  Version : 0.5.1
  Upstream Author : Antoine Bertin 
* URL : https://github.com/byroot/pysrt
* License : GPL-3
  Programming Lang: Python
  Description : Python parser for SubRip (srt) files

Pysrt can be used to edit or create SubRip (.srt) subtitle files from
Python. It supports parsing, shifting, removing, slicing, spliting and
rescaling of these files.

This library is needed as a dependency of subliminal (ITP #729781).
It will be maintained within the Debian Python Modules Team. 

A preliminary version provided by Oxan van Leeuwen can be found at the
following URL:

https://github.com/oxan/pysrt-debian

-- 
Etienne Millon


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



Bug#729870: ITP: guessit -- Python library for guessing information from video filenames

2013-11-18 Thread Etienne Millon
Package: wnpp
Severity: wishlist
Owner: Etienne Millon 
X-Debbugs-Cc: python-modules-t...@lists.alioth.debian.org

* Package name: guessit
  Version : 0.6.2
  Upstream Author : Antoine Bertin 
* URL : https://github.com/wackou/guessit
* License : LGPLv3
  Programming Lang: Python
  Description : Python library for guessing information from video filenames

GuessIt is a Python library that tries to extract as much information
as possible from a video file. It has a very powerful filename matcher
that allows to guess a lot of metadata from a video using only its
filename. This matcher works with both movies and TV show episodes.

This library is needed as a dependency of subliminal (ITP #729781).
It will be maintained within the Debian Python Modules Team. 

A preliminary version provided by Oxan van Leeuwen can be found at the
following URL:

https://github.com/oxan/guessit-debian

-- 
Etienne Millon


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



Bug#729869: ITP: babelfish -- Python library to work with countries and languages

2013-11-18 Thread Etienne Millon
Package: wnpp
Severity: wishlist
Owner: Etienne Millon 
X-Debbugs-Cc: python-modules-t...@lists.alioth.debian.org

* Package name: babelfish
  Version : 0.3.0
  Upstream Author : Antoine Bertin 
* URL : https://github.com/Diaoul/babelfish
* License : BSD-3
  Programming Lang: Python
  Description : Python library to work with countries and languages

Babelfish makes it easy to work with countries, languages, scripts,
ISO codes and IETF codes from Python. It has converters between all
different data can be extended to use custom converters and data.

This library is needed as a dependency of subliminal (ITP #729781).
It will be maintained within the Debian Python Modules Team. 

A preliminary version provided by Oxan van Leeuwen can be found at the
following URL:

https://github.com/oxan/babelfish-debian

-- 
Etienne Millon


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



Bug#729868: ITP: enzyme -- Python module to parse video metadata

2013-11-18 Thread Etienne Millon
Package: wnpp
Severity: wishlist
Owner: Etienne Millon 
X-Debbugs-Cc: python-modules-t...@lists.alioth.debian.org

* Package name: enzyme
  Version : 0.4.0
  Upstream Author : Antoine Bertin 
* URL : https://github.com/Diaoul/enzyme
* License : Apache2
  Programming Lang: Python
  Description : Python module to parse video metadata

Enzyme extracts metadata such as the type and encoding of video
streams, type, encoding and language of audio streams, chapters, tags
and subtitle tracks from video files.

This library is needed as a dependency of subliminal (ITP #729781).
It will be maintained within the Debian Python Modules Team.

A preliminary version provided by Oxan van Leeuwen can be found at the
following URL:

https://github.com/oxan/enzyme-debian

-- 
Etienne Millon


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



Bug#729781: ITP: subliminal -- Command-line tool to search and download subtitles

2013-11-17 Thread Etienne Millon
* Oxan van Leeuwen  [131117 13:29]:
> I've just finished and published my packaging of subliminal and its
> dependencies yesterday:
> - python-charade  https://github.com/oxan/charade-debian
> - python-enzyme   https://github.com/oxan/enzyme-debian
> - python-babelfishhttps://github.com/oxan/babelfish-debian
> - python-guessit  https://github.com/oxan/guessit-debian
> - python-pysrthttps://github.com/oxan/pysrt-debian
> - subliminal  https://github.com/oxan/subliminal-debian

Cool, thanks ! Feel free to file ITP bugs in advance to prevent duplication.
not happen again.

> Feel free to re-use (parts of) the packaging. I'm also a DPMT
> member, so I'm willing to help out with the packaging there later on
> too (I didn't push to DPMT initially because Alioth is down and I'm
> not sure whether I can personally invest enough time to keep them up
> to Debian quality standards).

Sure ! I'll see how I can reuse it, and add you to Uploaders if you're
fine with this.

-- 
Etienne Millon


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



Bug#729781: ITP: subliminal -- Command-line tool to search and download subtitles

2013-11-17 Thread Etienne Millon
Package: wnpp
Severity: wishlist
Owner: Etienne Millon 
X-Debbugs-Cc: python-modules-t...@lists.alioth.debian.org

* Package name: subliminal
  Version : 0.7.2
  Upstream Author : Antoine Bertin 
* URL : http://subliminal.readthedocs.org/
* License : MIT
  Programming Lang: Python
  Description : Command-line tool to search and download subtitles

It comes with an easy to use CLI (command-line interface) suitable for
direct use or cron jobs.
 
Subliminal uses multiple providers to give users a vast choice and
have a better chance to find the best matching subtitles. Providers
are extensible through a dedicated entry point.

I intend to maintain this package within the Debian Python Modules
Team.

It has a few unpackaged dependencies (I'll file RFPs and/or ITPs if
there is no objection to the inclusion of subliminal):

  - python-charade (ITP #698258, not sure if replaceable by chardet)
  - python-enzyme (same author ; https://github.com/Diaoul/enzyme ; Apache2)
  - python-babelfish (same author ; https://github.com/Diaoul/babelfish ; BSD-3)
  - python-guessit (https://github.com/wackou/guessit ; LGPLv3)
  - python-pysrt (https://github.com/byroot/pysrt ; GPLv3)

-- 
Etienne Millon


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



Bug#727781: zsnes: Fatal error when loading state (DSP-1 ROM)

2013-11-05 Thread Etienne Millon
Just an update.

The copying code makes some assumptions about the object code that
don't hold true with a "modern compiler" (it's hard to know for how
long it's been broken, though).

For example consider the following line:

> copy_func(&buffer, &Op02FX, 11*4+3*4+28*8+128);

The intent is probably to copy global variables as a block. The 11
first words refer to the following variables defined in zstate.c:
(2 bytes for each variable and 2 bytes of padding)

> short Op02FX;
> short Op02FY;
> short Op02FZ;
> short Op02LFE;
> short Op02LES;
> short Op02AAS;
> short Op02AZS;
> short Op02VOF;
> short Op02VVA;
> short Op02CX;
> short Op02CY;

(I don't really know what the 3*4+28*8 refer to, but there seem to be
128 bytes of padding after each block)

However, the memory layout is different than what's in the file:

> 0x890b208 : 0
> 0x890b20a : 0
> 0x890b20c : 0
> 0x890b20e :  0
> 0x890b210 :  0
> 0x890b212 :  0
> 0x890b214 :  0
> 0x890b216 : 0
> 0x890b218 :  0

(BTW if someone knows how to print the above global layout in an
automated way, I'm interested. The above is just done by hand with
gdb)

The solution of copying variable by variable does surprisingly not
work, presumably because some data originally copied through those
bytes of padding (accessed Op02FX, Op0AVS and Op06X) has a different
layout.

However, keeping this hack for only Op02FX allows loading to work,
at least in my tests. But some state may be missed.

The "good" solution would be to refactor the code to use a global
struct for DSP1 state but this is a bit of work. Interestingly enough,
dsp1proc.asm internally uses a 16 word DSP1VARS "structure" that may
be copied instead.

-- 
Etienne Millon


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



Bug#727781: zsnes: Fatal error when loading state (DSP-1 ROM)

2013-10-26 Thread Etienne Millon
Package: zsnes
Version: 1.510+bz2-6
Severity: normal

Hi,

While loading a ROM I noticed that a fatal error due to FORTIFY_SOURCE
made zsnes abort with the following error message:

*** buffer overflow detected ***: /usr/bin/zsnes terminated

(see also #712790 for the general case)

After further investigation it seems that the following line is the
culprit (zstate.c +189):

copy_func(&buffer, &DSP1COp, 70+128);

The game uses indeed the DSP-1 coprocessor, and DSP1COp is declared as
a "extern unsigned char" in C (and "resb 1" in asm) but is used to
address the next variables.

I'm working on a patch to address this particular issue, but other
coprocessors probably use a similar system to load states.

-- 
Etienne Millon


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



  1   2   3   4   >