[Rpm-maint] Patch: Add support of syslog when installing/removing packages
skvidal at fedoraproject.org
Thu Mar 3 15:36:40 UTC 2011
On Thu, 2011-03-03 at 17:28 +0200, Ville Skyttä wrote:
> On 03/03/2011 01:44 PM, Panu Matilainen wrote:
> > Some (stylistic) issues with the patch though:
> A couple of other thoughts, perhaps for future development:
> Add some way of specifying the syslog ident string instead of hardcoding
> it. That way logging from for example those upper level depsolvers
> could show up as $depsolvername in the syslog instead of rpm.
> Use "rpm", not "[RPM]" for the ident string by default.
> Add a process id to the ident string.
> Log arch as well as NVR.
> So the three last thoughts combined would result in log entries like:
> Feb 27 11:25:51 $hostname rpm[$pid]: firefox-4.0-0.b12.4.mga1.x86_64
> ...or $depsolver[$pid] instead of rpm[$pid] if the ident string was set
> to $depsolver.
Seems like if you're going to go through the trouble of adding this
you'll want something you can match reliably on.
rpm[$pid:[$processname|$depsolvername]] pkgname pkgstate/action
then again there is a massive truckload of additional info you'd want in
More information about the Rpm-maint