Bug#457855: python-debian: PackageFile example crashes looking for unexisting field in package

[Adeodato Simó]

Hello. I've addressed these three bug reports with:

  http://lists.alioth.debian.org/pipermail/pkg-python-debian-commits/2007-December/000062.html

Thanks for reporting! (I didn't use your patches, sorry for that -- the
rationale is explained in debian/changelog in the above diff.)

* Antonio Terceiro [Wed, 26 Dec 2007 11:38:49 -0300]:

> Not all entries in /var/lib/dpkg/status have a Maintainer field. The
> attached patch makes the example work as intended (AFAICT).


* Antonio Terceiro [Wed, 26 Dec 2007 11:31:32 -0300]:

> the PackageFile example currently crashes with a TypeError:

> terceiro at morere:~$ python /usr/share/doc/python-debian/examples/packages/grep-maintainer terceiro
> Traceback (most recent call last):
>   File "/usr/share/doc/python-debian/examples/packages/grep-maintainer", line 26, in ?
>     if maint_RE.search(pkg['maintainer']):
> TypeError: list indices must be integers

> This happens because the __yield__ method in PackageFile class has two yield
> statements, one 'yield pkg' and other 'yield dict(pkg)'.

> The attached patch fixes the problem by putting dict() around 'pkg' in the
> first yield.


* Mike O'Connor [Mon, 24 Dec 2007 11:45:45 -0500]:

> When I try to use debian_bundle.debian_support.updateFile, I get the
> following exception:

> stew at tang:~$ /tmp/test_updateFile.py
> Traceback (most recent call last):
>   File "/tmp/test_updateFile.py", line 9, in ?
>     updateFile( remote, local, True )
>   File "/var/lib/python-support/python2.4/debian_bundle/debian_support.py", line 399, in updateFile
>     for (field, value) in fields:
> ValueError: too many values to unpack

> Attached are two files, test_updateFile.py is a small test case that
> demonstrates the problem. fix_updateFile.diff is a patch which fixes
> this problem.

-- 
Adeodato Simó                                     dato at net.com.org.es
Debian Developer                                  adeodato at debian.org
 
- For the future: I don't really appreciate comments about my hair.
- Oh, I'm sorry. I was just trying to make conversation. And I assumed
  you didn't want me to go anywhere near your eyebrows.
                -- Lori Colson and Catherine Piper (Boston Legal)