Merge lp:~thekorn/aptdaemon/fix-702217-dbus_struct_TypeError into lp:aptdaemon

[Markus Korn]

dbus.Struct does not allow in-pace changes of values. This branch fixes bug 702217 by using the same workaround which is also used a few lines later in the code.

[Sebastian Heinlein (glatzor)]

Thanks for your patch. A fix was already uploaded to the trunk branch and will land the next days in natty.

[Sebastian Heinlein (glatzor)]

review: Disapprove

[Markus Korn (thekorn)]

Ok, sorry for the noise than.
BUT, when I `sudo python setup.py install` lp:aptdeamon rev577 and restart aptd I still get this TypeError.

[Sebastian Heinlein (glatzor)]

On Tue, 08 Feb 2011 11:29:10 -0000, Markus Korn <email address hidden> wrote:
> Ok, sorry for the noise than.
> BUT, when I `sudo python setup.py install` lp:aptdeamon rev577 and
restart
> aptd I still get this TypeError.

I would recommend to not install aptdaemon this way.

You can run aptdaemon from the source repository:

bzr branch lp:aptdaemon
cd aptdaemon
sudo ./aptd -td --replace

[Michael Vogt (mvo)]

Sebastian fixed it in your way now in trunk (and added a test along the way).

[Michael Vogt (mvo)]

Thanks :)

=== modified file 'aptdaemon/worker.py'
--- aptdaemon/worker.py	2011-02-08 08:10:01 +0000
+++ aptdaemon/worker.py	2011-02-08 10:27:19 +0000
@@ -1000,7 +1000,7 @@
             try:
                 pkg = self._cache[deb.pkgname]
             except KeyError:
-                trans.packages[PKGS_INSTALL] = [deb.pkgname]
+                trans.packages = [[deb.pkgname], [], [], [], [], []]
             else:
                 if pkg.is_installed:
                     # if we failed to get the size from the deb file do nor