summaryrefslogtreecommitdiffstats
path: root/src/lib/run_event.c
Commit message (Collapse)AuthorAgeFilesLines
* wizard: hook selection checkboxes to reporters.Denys Vlasenko2011-06-011-1/+1
| | | | | | | | | | | | | | | | | | | | | Checkboxes are controlled by the following elements in .xml: <requires-items> ITEM1,ITEM2 </requires-items> <exclude-items-by-default> ITEM1,ITEM2 </exclude-items-by-default> <exclude-items-always> ITEM1,ITEM2 </exclude-items-always> <exclude-binary-items> yes / no </exclude-binary-items> <include-items-by-default> ITEM1,ITEM2 </include-items-by-default> exclude-items-by-default and exclude-items-always can be "*" meaning "all". include-items-by-default specifies which items are included (checked) is exclude-items-by-default is "*". Else, all are checked by default. the set of iters NOT included is passed in $EXCLUDE_FROM_REPORT variable. bugzilla, kerneloops, mailx, print and rhtsupport are made aware of this variable. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: support VAR~=REGEX matchDenys Vlasenko2011-05-231-2/+37
| | | | | | | This makes in possible to configure analyze_xsession_errors to appear only for X-related stuff Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: add comment. No code changesDenys Vlasenko2011-05-231-0/+7
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: explain pop_next_command, move free_rule_list inside it.Denys Vlasenko2011-05-171-4/+13
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event.c: fix "EVENT=post-create component=mypkg doesn't work" bug. ↵Denys Vlasenko2011-05-171-234/+200
| | | | | | Closes bz#531365 Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* smart_event.conf: suppress stray output of "which" commandDenys Vlasenko2011-05-171-4/+2
| | | | | | Also, fix some typos in comments Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* mass replace of crash_data with problem_dataDenys Vlasenko2011-04-231-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: prevent infinite recursion in include directives in abrt_event.confDenys Vlasenko2011-04-211-5/+11
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* gui-wizard-gtk: add load_event_config_data() to wizardDenys Vlasenko2011-03-091-16/+16
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* make fork_execv_on_steroids capable of setting env vars tooDenys Vlasenko2011-03-081-5/+9
| | | | | | Before, it could only unset them. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: load_event_config can be done with DD_OPEN_READONLYDenys Vlasenko2011-03-031-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* gui-wizard-gtk: fix stealing againDenys Vlasenko2011-03-031-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* dump_dir API: rename dd_dir field to dd_dirnameDenys Vlasenko2011-02-281-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* gui-wizard-gtk: run multiple reporters if they are selectedDenys Vlasenko2011-02-231-0/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: add async run event machineryDenys Vlasenko2011-02-181-83/+157
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* use run_state->children_count == 0 check for "event not conf'd" conditionDenys Vlasenko2011-02-011-1/+1
| | | | | | | | | | | | run_event_on_FOO() was returning -1 when it found that not even one program was run on the requested event. Which is not a very natural return value: in many cases this isn't an error. This change makes it return 0 in this case. It is ok since now we have run_state->children_count member which can be checked for 0 by those callers which want to detect this condition. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Message text and other small cosmetic fixesDenys Vlasenko2011-01-281-1/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* make list_possible_events return empty on non-dump directoriesDenys Vlasenko2011-01-281-4/+12
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* run_event: add children_countDenys Vlasenko2011-01-271-2/+14
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* preparatory changes for abrt-cli local processing changeDenys Vlasenko2011-01-271-1/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt_event.conf parser is updated to handleDenys Vlasenko2011-01-131-16/+139
| | | | | | | | | | | | | | | "include GLOB_PATTERN" line which expands GLOB_PATTERN and recurses into every matching file. Another small change makes it possible to have one rule to span lines: our current default abrt_event.conf has sosreport invocation which looks very ugly. Sosreport invocation is rewritten in a much more readable multi-line form now. The comment in src/daemon/abrt_event.conf is modified to explain the recognized syntax. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* extend run_event() to run_event_on_dir_name() and run_event_on_crash_data()Denys Vlasenko2010-12-221-1/+20
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* factor out headers so that "report lib" API is separated out of abrt APIDenys Vlasenko2010-12-011-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch minimally affects code per se - it adds create_crash_dump_dir() function which takes in-memory representation of the dump (map_crash_data_t object) and creates an on-disk representation. Then it returns pointer to struct dump_dir. With this function, it is possible to run an event on a user-constructed map_crash_data_t: map_crash_data_t cd; add_to_crash_data(cd, "foo", "bar"); struct dump_dir *dd = create_crash_dump_dir(cd); char *dir_name = strdup(dd->dd_dir); dd_close(dd); run_event(run_state, dir_name, event); delete_crash_dump_dir(dir_name); which is, basically, what report library is about. The biggest part of the patch is reshuffling of header files, with the following result: three header files which are not cluttered by other ABRT stuff, and expose the following API each: crash_dump.h - in-memory crash dump data structs and ops dump_dir.h - on-disk crash dump data structs and ops run_event.h - run_event() and friends These is a test application, test_report.cpp.cpp, which demonstrates (and tests) usage of these headers. (grep for "test-report" and enable it in build system if you want it to be built). It creates a temporary dump in /var/run/abrt/tmp-NNNN-NNNNN and runs "report" event. Deleting of temp dump is disabled for testing purposes - you might want to see the contents. Here is how the binary looks like: text data bss dec hex filename 3730 668 48 4446 115e /usr/bin/test-report linux-vdso.so.1 => (0x00007ffffa80f000) libabrt.so.0 => /usr/lib64/libabrt.so.0 (0x00007fad1f35e000) libglib-2.0.so.0 => /lib64/libglib-2.0.so.0 (0x0000003c1f200000) libstdc++.so.6 => /usr/lib64/libstdc++.so.6 (0x0000003c2c200000) libm.so.6 => /lib64/libm.so.6 (0x00007fad1f0da000) libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x0000003c28e00000) libc.so.6 => /lib64/libc.so.6 (0x00007fad1ed5b000) /lib64/ld-linux-x86-64.so.2 (0x00007fad1f570000) Next step would be to clean up the namespace, then rename libabrt.so into libreport.so, then split it into a separate package so that we can install it without installing other two abrt .so files. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrtd: suppress spurious warning about missing coredump and inform_allDenys Vlasenko2010-11-251-1/+1
| | | | | | Also plugs two memory leaks Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Make "abrt-handle-crashdump -e BOGUS_EVENT" fail (also fail if asked over dbus)Denys Vlasenko2010-11-151-7/+5
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move inc/ and lib/ to src/. No code changesDenys Vlasenko2010-11-151-0/+276
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>