[Rpm-maint] [rpm-software-management/rpm] Extend dbus-announce plugin (PR #3532)
mcrha
notifications at github.com
Thu Jan 23 09:22:30 UTC 2025
The first commit only adds an existing argument (`res`) into the D-Bus message - I suppose it had been intended to provide a transaction result as well.
The second commit adds also information about the changes, like the operation and the NEVRA of the transaction packages. This implementation can have one downside, with many packages the D-Bus message will be large. I can add some limit, like up to 100 packages or so, and pass an empty array in such case. Using file descriptors feels like an overhead for this information.
----------
My intention is to use the details about the transaction in the gnome-software, to update its GUI only if any of the touched packages corresponds to an application covered by the app itself. While it's relatively easy to get to the list of the installed packages, the removals are not recorded, those transactions are lost immediately (I guess, when a transaction does both install and remove, then only install is recorded anyway). This can have a significant performance benefit for the gnome-software, because currently it can only tell the app to "reload everything", which is a lot of work for the CPU & co..
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/3532
-- Commit Summary --
* dbus_announce: Pass also transaction result over D-Bus
* dbus_announce: Provide information about packages in the transaction
-- File Changes --
M plugins/dbus_announce.c (59)
-- Patch Links --
https://github.com/rpm-software-management/rpm/pull/3532.patch
https://github.com/rpm-software-management/rpm/pull/3532.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3532
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3532 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250123/da4421c5/attachment-0001.htm>
More information about the Rpm-maint
mailing list