[Rpm-maint] [rpm-software-management/rpm] Extend dbus-announce plugin (PR #3532)

[Michal Domonkos]

dmnks left a comment (rpm-software-management/rpm#3532)

> I'm also thinking, wouldn't it be cleaner to just split `send_ts_message()` into two functions, one for the existing signals and one for the extended/detailed ones? That way, we wouldn't have to e.g. reuse the static `name` string and check its length.

Hmm, the "detailed" signal includes the same payload as the existing variant (`dbcookie` and `tid`) so another option would be keeping `send_ts_message()` but adding a boolean parameter for controlling whether the detailed signal should be sent instead.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3532#issuecomment-2678190990
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3532/c2678190990 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250224/3f87bbe5/attachment.htm>