summaryrefslogtreecommitdiffstats
path: root/src/daemon/abrt-action-save-package-data.cpp
Commit message (Collapse)AuthorAgeFilesLines
* make #includes simplerDenys Vlasenko2010-11-051-1/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* introduce abrt_action.conf; use it for post-create stepDenys Vlasenko2010-11-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This patch replaces only the post-create step with new logic (the step which happens when abrtd detects fresh crash dump dir and needs to decide what to do with it), but it turns out this step is one of hard ones: it needs special handling of UUID. So a good chunk of hard-ish stuff is already in this patch. It also contains logic to collect the log from actions, even though so far it is simply logged. Other steps (like reporting step) will pipe it to clients. post-create step simply has no client to pipe output to. But the code is there already. GetLocalUUID() members in all plugins are unused now and are deleted from all classes. Next patches will move run_event() function into libABRT, so that it can be used from e.g. abrt-handle-crashdump utility if/when we will write it, from clients directly and so on. For now, it lives in MiddleWare.cpp since it is only used there. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* reduce usage of daemon's library in non-daemon partsDenys Vlasenko2010-10-251-0/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* restore correct handling of argv[1] of scriptsDenys Vlasenko2010-10-221-24/+21
| | | | | | Bad boy Karel "improved" it incorrectly, and did not revert it. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* change dd_opendir API to return dd pointer (no need to dd_init it separately)Denys Vlasenko2010-10-151-6/+6
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* restore abrt-action-save-package-data's description in --helpDenys Vlasenko2010-10-131-4/+9
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Add a warning comment that options and their enums should be in syncDenys Vlasenko2010-10-121-5/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Return a bitmask of found options from parse_opts()Denys Vlasenko2010-10-121-15/+12
| | | | | | | | | | | | | This patch makes it easier to analyze "bool" type options - now they are all just bits in a single return value, no need to have an integer variable for each option. The previous behavior is retained too: it is useful for "cumulative" options like -vvv. The patch also removes all OPT_GROUPs. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Remove arrays in usage text, use a single string insteadDenys Vlasenko2010-10-081-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | Currently, usage is an array of pointers to string. It is meant to support this: usage: foo --bar or: foo --baz or: foo -z The patch removes this and makes usage just a string: usage: foo --bar Why remove this? (1) We can achieve old format by using the usage string with many lines: usage = "foo --bar\n" " or: foo --baz\n" " or: foo -z"; (2) We don't use multi-line usage anyway. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* add --help option in parse_opts(), not at every call site.Denys Vlasenko2010-10-081-8/+7
| | | | | | | This patch removes the need to add --help option to every program. It is added (and handled) by parse_opts(). Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* use OPT_GROUP macro for help optionNikola Pajkovsky2010-10-081-0/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* use OPT__VERBOSE macro for help optionNikola Pajkovsky2010-10-081-1/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* use OPT__HELP macro for help optionNikola Pajkovsky2010-10-081-1/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* abrt-action-save-package-data uses new parserNikola Pajkovsky2010-10-071-33/+26
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* g_settings_setBlackListedPaths uses GListNikola Pajkovsky2010-09-241-4/+2
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* g_settings_setBlackListedPkgs uses GListNikola Pajkovsky2010-09-241-3/+7
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* g_settings_setOpenGPGPublicKeys uses GListNikola Pajkovsky2010-09-241-4/+3
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* This patch splits off abrtd's package/component/descriptionDenys Vlasenko2010-09-221-0/+332
generation into a separate tool: abrt-action-save-package-data: invalid option -- 'z' Usage: abrt-action-save-package-data -d DIR [-vs] Query package database and save package name, component, and description Options: -d DIR Crash dump directory -v Verbose -s Log to syslog This also allows for debugging and regression testing of abrt-action-save-package-data - it can be simply run from command-line. Also it provides a better fault isolation - crash in abrt-action-save-package-data does not take down abrtd. Same goes for isolation of memory leaks - old code actually had leaks on a rarely used error path. New code doesn't, but if it would have, it wouldn't matter, since the process is short-lived. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>