[packagekit] Changing the Package signal
Ken VanDine
ken at vandine.org
Sun Sep 30 19:21:44 PDT 2007
When I updated the search method, returning "installed" seems to give
the opposite result in the UI. And same for "available". I suspect
the UI has it turned around... or I hope. But my C is to weak to see
the fix.
--Ken
On 9/30/07, Richard Hughes <hughsient at gmail.com> wrote:
> >From TODO: convert the magic number in Package to a string type
>
> I've just committed to master a change in the callback of Signal. The
> old callback relied on a magic number to mean different things for
> different situations, and was very fragile and against the spirit of the
> dbus interface as it was not self documenting.
>
> The new callback is exactly the same, but uses enumerated types. You can
> see what I mean looking at the commit
> 50f55e02c2bd35a39b5425f08eec2e5d94223606 - basically the compiled
> backends just use PK_INFO_ENUM_INSTALLED rather than "1" and the helper
> backends now have to return "available" rather than "0".
>
> I've converted the compiled backends (still, probably a good idea to
> check) and had a stab at the yum backend (probably wrong) - I've not
> touched the conary backend as it looked more complicated. It will
> probably take about 2 minutes to convert this - sorry - my python skills
> are very limited.
>
> So basically, the old mapping to new mapping is thus:
>
> When searching:
> 1 installed PK_INFO_ENUM_INSTALLED
> 0 available PK_INFO_ENUM_AVAILABLE
>
> When getting updates:
> 0 normal PK_INFO_ENUM_NORMAL
> 1 security PK_INFO_ENUM_SECURITY
> 2 important PK_INFO_ENUM_IMPORTANT
> 3 low PK_INFO_ENUM_LOW
>
> When processing packages:
> 0 downloading PK_INFO_ENUM_DOWNLOADING
> 1 installing PK_INFO_ENUM_INSTALLING
> x removing PK_INFO_ENUM_REMOVING
>
> Ken/Elliot - can you yell when you've made the few changes to the conary
> backend, and also Tim/Whoever for the yum backend. You can use the
> INFO_* defines I've added in packagekit.py. When these are done I'll
> push to fd.o.
>
> Thanks,
>
> Richard.
>
> p.s. this is hopefully the last API change before 0.1.0 - there will be
> one more signal added, but nothing that requires backend changes.
>
>
> _______________________________________________
> PackageKit mailing list
> PackageKit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/packagekit
>
--
Ken VanDine
http://ken.vandine.org
More information about the PackageKit
mailing list