Package: release.debian.org Severity: normal User: release.debian....@packages.debian.org Usertags: unblock
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 Please unblock package weboob This is an NMU to fix bug #690301 which render makes many use cases of weboob fail. The change is a small patch cherry-picked from upstream, which works around an incompatible API change of python-prettytable. The debdiff also includes a small typo fix which was made by the package maintainer in his Git repository. I know it should not be here strictly speaking, so tell me if I should prepare a version without it. unblock weboob/0.c-4.1 - -- System Information: Debian Release: wheezy/sid APT prefers testing APT policy: (990, 'testing'), (500, 'unstable'), (500, 'stable'), (1, 'experimental') Architecture: amd64 (x86_64) Kernel: Linux 3.2.0-3-amd64 (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 -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iQIcBAEBCgAGBQJQgE/pAAoJEOryzVHFAGgZHO0QAJumQyIy7V62O79jQAV7GDnG ZD2qmZE/RUzocNt8iv7l1is7yxnOm9+jbg1WuKKUfnYgY9E1fN6LRwKwFr/YbyBK bRz8wF44MG+te+tCDUPVMq6WvaOZrFkCEE3CXaOplVJ4d7PX6P72TJSsv+yZUpC4 QOGRFMst2/pgYdT2a75zVMhOn+toISgvK07eq47nqIHeZqtj1uOHLOOuPA+m2G8Q 5XoBjlgBYRzg80JqMu9BAIKEnUGzGCm3c0U1WznC48OKP8WS+0W9O07WcLLGoARP S5/Itv1A1+HGVoXgCUNnjzV5z4o2Maa1keUtCq6ZmPr2HIzuJWhhwxc9abEsGZQB P/tQpF8VJrs2TggqURPEYMHxEiDCougj7MzfHug6NEL7lI5IDOwGW4V+aOye5Kqz 8uSS57Fcxq62/sGOTLWvLCvEs+Ri5a7GzWHb+HLF/lkIU6r40KkRfJKSwmXZwTzX wO4sgB/k+i1IR3FlEbUq6xA7HDKaCe7ysB9Xks4DBJIzcsxVUiFwJjWIk7E8XqjT RUnZK0rojMywWcWjCFzdHVKjbYZVw15w+Q15eW18PsPsx8HROAzkR6B/Kx0m0YbN YTu1uHZdSZTLIgk+O+xgbUyo+F/Kd0NPUALtwWGd5nXQvWbCOKBVy+0+bSmISUq4 fjNiuio2Kvhtx3ChnlFx =s1rT -----END PGP SIGNATURE-----
diff -Nru weboob-0.c/debian/changelog weboob-0.c/debian/changelog --- weboob-0.c/debian/changelog 2012-06-13 11:43:34.000000000 +0200 +++ weboob-0.c/debian/changelog 2012-10-18 20:37:01.000000000 +0200 @@ -1,3 +1,11 @@ +weboob (0.c-4.1) unstable; urgency=low + + * Non-maintainer upload, already approved by the maintainer. + * Fix incompatibility with the last version of python-prettytable. + (Closes: #690301) + + -- Tanguy Ortolo <tanguy+deb...@ortolo.eu> Thu, 18 Oct 2012 20:25:14 +0200 + weboob (0.c-4) unstable; urgency=low * Add watch file for uscan @@ -8,8 +16,8 @@ * Fix typo and whatis entry in manpages * Ignore dynamic quilt files * Fix call to dh_python2 - * Bump debhelper compat level to 9 (recommanded version) - * Fail when not all files are installed or explicitely ignored + * Bump debhelper compat level to 9 (recommended version) + * Fail when not all files are installed or explicitly ignored * Install bash autocompletion * Add weboob-debug to python-weboob-core package * Do not install egg file, only meta-data diff -Nru weboob-0.c/debian/patches/series weboob-0.c/debian/patches/series --- weboob-0.c/debian/patches/series 2012-06-11 00:02:37.000000000 +0200 +++ weboob-0.c/debian/patches/series 2012-10-18 20:12:03.000000000 +0200 @@ -1,2 +1,3 @@ for-upstream_fix-manpage-header.patch for-upstream_fix-manpage-typos.patch +workaround-prettytable-api.patch diff -Nru weboob-0.c/debian/patches/workaround-prettytable-api.patch weboob-0.c/debian/patches/workaround-prettytable-api.patch --- weboob-0.c/debian/patches/workaround-prettytable-api.patch 1970-01-01 01:00:00.000000000 +0100 +++ weboob-0.c/debian/patches/workaround-prettytable-api.patch 2012-10-18 20:20:41.000000000 +0200 @@ -0,0 +1,30 @@ +From: Florent <web...@flo.fourcot.fr> +Origin: upstream, http://git.symlink.me/?p=romain/weboob-stable.git;a=commit;h=a8e2342edcbc38b940089822dafdcebd27b00946 +Bug-Debian: http://bugs.debian.org/690301 +Last-Update: 2012-10-18 +Subject: Fix a frequent crash due to an API change in table formater + +This patch works around the incompatible API change in python-prettytable +reported in bug #673790. +diff --git a/weboob/tools/application/formatters/table.py b/weboob/tools/application/formatters/table.py +index d11dc11..55e7468 100644 +--- a/weboob/tools/application/formatters/table.py ++++ b/weboob/tools/application/formatters/table.py +@@ -69,7 +69,13 @@ class TableFormatter(IFormatter): + s += "\n" + table = PrettyTable(list(column_headers)) + for column_header in column_headers: +- table.set_field_align(column_header, 'l') ++ # API changed in python-prettytable. The try/except is a bad hack to support both versions ++ # Note: two versions are not exactly the same... ++ # (first one: header in center. Second one: left align for header too) ++ try: ++ table.set_field_align(column_header, 'l') ++ except: ++ table.align[column_header] = 'l' + for line in queue: + table.add_row(line) + +-- +1.7.2.5 +