[Rpm-maint] Patch: Add support of syslog when installing/removing packages
seth vidal
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
> installed
>
> ...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.
so maybe:
rpm[$pid:[$processname|$depsolvername]] pkgname pkgstate/action
then again there is a massive truckload of additional info you'd want in
a log.
-sv
More information about the Rpm-maint
mailing list