On Thu, 2011-08-04 at 18:08 +0200, Nikola Pajkovsky wrote:
Signed-off-by: Nikola Pajkovsky npajkovs@redhat.com
abrt-oops.init | 2 +- src/plugins/Makefile.am | 3 ++- src/plugins/abrt-dump-oops.c | 17 ++++++++++++++++- src/plugins/abrt-oops.service | 2 +- 4 files changed, 20 insertions(+), 4 deletions(-)
Looks good.
const char *program_usage_string = _( "\b [-vsrowx] [-d DIR] FILE\n"
" or: \b [-vsrowx] -D file\n"
Small nit: file -> FILE
OPT_STRING('d', NULL, &debug_dumps_dir, "DIR", _("Create ABRT dump in DIR for every oops found")), OPT_BOOL( 'x', NULL, NULL, _("Make the dump directory world readable")),
OPT_BOOL( 'D', NULL, NULL, _("Load dynamically DUMP_DIR location from abrt.conf")),
I think this explanation is confusing. How about "Same as -d DumpLocation, DumpLocation is specified in abrt.conf"