----- Original Message ----- From: "Denys Vlasenko" <[email protected]> To: [email protected] Cc: "Jakub Filak" <[email protected]> Sent: Monday, December 3, 2012 4:26:56 PM Subject: Re: [ABRT PATCH 1/3] add Autoreporting configuration options
On 11/28/2012 06:39 PM, Jakub Filak wrote: > - related to trac#763, trac#768 > > Signed-off-by: Jakub Filak <[email protected]> > --- > src/daemon/abrt.conf | 15 +++++++++++++++ > src/include/libabrt.h | 4 ++++ > src/lib/abrt_conf.c | 18 ++++++++++++++++++ > 3 files changed, 37 insertions(+) > > diff --git a/src/daemon/abrt.conf b/src/daemon/abrt.conf > index c7ad417..6cc3376 100644 > --- a/src/daemon/abrt.conf > +++ b/src/daemon/abrt.conf > @@ -20,3 +20,18 @@ MaxCrashReportsSize = 1000 > # selinux policy. > # > DeleteUploaded = no > + > +# A name of event which is run automatically after problem's detection. The > +# event should perform some fast analysis and print out THANKYOU message if > the > +# problem is konwn. > +# > +# In order to run this event automatically after detection, the > +# AutoreportingEnabled option must be configured to 'yes' > +# > +# Default value: report_uReport > +# > +AutoreportingEvent = report_uReport > + > +# Enables automatic running of the event configured in AutoreportingEvent > option. > +# > +AutoreportingEnabled = no How about having only one directive, AutoreportingEvent, and if it is not set, then we assume autoreporting should not be done? (IOW: AutoreportingEnabled directive seems to be superfluous). Seems to be superfluous here but makes sense with the next patch. User can enable autoreporting in $XDG_HOME/.config/abrt/settings/*.conf by setting AutoreportingEnabled to 'yes' without knowledge of an event which he want to run.
