[packagekit] Allowing multiple actions in a transaction

Richard Hughes hughsient at gmail.com
Sun Mar 9 17:45:15 PDT 2008


On Sun, 2008-03-09 at 20:03 +0000, Richard Hughes wrote:
> We'll do this API break slowly, as I appreciate it's a large internal
> and external API change that great potential to go wrong. ;-)

Well, just for one method (UpdatePackages) this is the diffstat:

 backends/dummy/pk-backend-dummy.c       |   16 ++++++--
 backends/test/pk-backend-test-dbus.c    |    2 -
 backends/test/pk-backend-test-fail.c    |    6 +--
 backends/test/pk-backend-test-succeed.c |    6 +--
 backends/yum/pk-backend-yum.c           |   14 +++++--
 backends/yum2/pk-backend-yum2.c         |    8 ++--
 client/pk-console.c                     |    2 -
 libpackagekit/pk-client.c               |   59 ++++++++++++++++++++++++--------
 libpackagekit/pk-client.h               |    7 ++-
 src/pk-backend-dbus.c                   |   10 ++---
 src/pk-backend-dbus.h                   |    4 +-
 src/pk-backend.h                        |    6 +--
 src/pk-engine.c                         |   25 +++++--------
 src/pk-engine.h                         |    4 +-
 src/pk-interface.xml                    |    4 +-
 src/pk-runner.c                         |   18 +++++----
 src/pk-runner.h                         |    4 +-
 17 files changed, 122 insertions(+), 73 deletions(-)

So, not a disaster. I've not even tried to convert the other backends,
as this is just proof of concept, but the attached patch shows what I
had to do. Nothing is in git yet. Comments welcome.

Richard.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: pk-update-to-updates.patch
Type: text/x-patch
Size: 25483 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/packagekit/attachments/20080310/b9704796/attachment-0004.bin>


More information about the PackageKit mailing list