summaryrefslogtreecommitdiffstats
path: root/src/plugins/abrt-action-bugzilla.cpp
Commit message (Collapse)AuthorAgeFilesLines
* abrt-action-bugzilla.cpp -> src/plugins/abrt-action-bugzilla.cNikola Pajkovsky2011-05-121-340/+0
| | | | | | no code change Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* xmlrpc and bugzilla in a new C coatNikola Pajkovsky2011-05-121-693/+75
| | | | | | | | | | | Bugzilla is splitted into two parts. One of them is core of bugzilla and second one is small module called librhbz. I did it, because I want to add a new functionality into bugzilla and I don't want to squash it into one big module. Librhbz is not a part of our libreport, because almost every function dies when feels badly (print error msg and then die). Server sends error or abrt sends corrupted xml or server sends corrupted xml. Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* move $ABRT_foo checking and setting into helper routines.Denys Vlasenko2011-05-041-11/+3
| | | | | | | I'm sick of fixing incorrect PROGNAME defines. Now g_progname is a variable and it is set from argv[0]. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Typo fix in commentDenys Vlasenko2011-04-291-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Logger: output "END:" line to delimit appended reports. closes bz#698458Denys Vlasenko2011-04-281-1/+3
| | | | | | Also changes Bugzilla and abrt-cli output format. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* mass replace of crash_data with problem_dataDenys Vlasenko2011-04-231-32/+32
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* GList was not set to NULL, possibly sigsegv on f15Nikola Pajkovsky2011-04-011-0/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* remove misleading error messageNikola Pajkovsky2011-03-301-3/+1
| | | | | | | | | | | | solution isn't good enough, but truly remove misleading message. Imagine the situation when user manually set environment Bugzilla_Password and manually run command a-a-b -c /path/to/config1 -c /path/config2 -c /path/config3. Now a-a-b smash all configs into one giant hash table. Just stir the pot. Environment can be set by cli and gui. We have a lot of input how to set config and question is how to determinate which config is wrong and blame someone/something. Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* Ticket #160 Hash is not present in BugzillaNikola Pajkovsky2011-03-281-1/+10
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* make $ABRT_PROG_PREFIX control whether log messages are prefixedDenys Vlasenko2011-03-251-15/+10
| | | | | | | | | | by abrt-action-foo programs. abrtd, abrt-server and abrt-handle-crashdump now have option -p which sets ABRT_PROG_PREFIX=1 for their children and thus makes them emit program name prefixes. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Replace FILENAME_MESSAGE with FILENAME_REPORTED_TO.Denys Vlasenko2011-03-181-1/+10
| | | | | | | | | | | This fixes the problem with abrt-cli not showing unreported bugs and abrt-gui not showing "green lamp" for reported bugs. In general, it introduces a way to record reporting results, such as Bugzilla URL, in dump dir. In the future, this enables such functions as "check BZ status". Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Merge branch 'bz'Nikola Pajkovsky2011-03-161-24/+6
|\ | | | | | | | | * bz: get rid of std::vector from bugzilla
| * get rid of std::vector from bugzillaNikola Pajkovsky2011-03-151-24/+6
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* | bugzilla reporter fixesDenys Vlasenko2011-03-161-23/+24
|/ | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-action-bugzilla: add compat code for old value for FILENAME_OS_RELEASEDenys Vlasenko2011-03-031-0/+6
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* remove FILENAME_REPRODUCE field. FILENAME_COMMENT should be used insteadDenys Vlasenko2011-03-031-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* get rid of FILENAME_DESCRIPTION, rename "release" to "os_release"Denys Vlasenko2011-02-101-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* split parse_release() into Bz and RHTS versionsDenys Vlasenko2011-01-251-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* plug memory leak on error pathsDenys Vlasenko2011-01-251-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* add OPT_LIST mechanism to handle multiply occurring optionsDenys Vlasenko2011-01-201-37/+34
| | | | | | | This allowed to convert abrt-action-bugzilla, abrt-action-rhtsupport and abrt-action-kerneloops to parse_opts() Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* make option handling more regular across all toolsDenys Vlasenko2011-01-201-5/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* trivial: more uniform -v option helpDenys Vlasenko2011-01-131-1/+1
| | | | 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/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* *: rename *crash_dump.* -> *crash_data.*Denys Vlasenko2010-12-221-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Rename foo_crash_dump_dir -> foo_dump_dirDenys Vlasenko2010-12-151-1/+1
| | | | | | | | | | | | | To be exact, these three functions are renamed: load_crash_data_from_crash_dump_dir create_crash_dump_dir delete_crash_dump_dir Rationale: data structure is called "struct dump_dir", not "struct crash_dump_dir" Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* add abrt_ prefixes to abrt-internal functions in libabrt.soDenys Vlasenko2010-12-081-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* removal of C++isms from libabrt, part 2Denys Vlasenko2010-12-081-3/+7
| | | | | | This patch rewrites a few places where we use C++ strings Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* removal of C++isms from libabrt, part 1Denys Vlasenko2010-12-081-36/+21
| | | | | | | | | | | | | This patch converts libabrt usage of C++ map<string, string> to a glib-based container, GHashTable. It is typedef-ed to map_string_h. We can't typedef it to map_string_t, since other parts of ABRT (daemon, cli) still use that name for C++ container. Also, exceptions are removed everywhere. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* This patch changes crash data to use C structures.Denys Vlasenko2010-12-061-35/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The smallest data element is: struct crash_item { char *content; unsigned flags; }; where content is, eh, content, and flags is a bit flag field. crash_data_t is a map of crash_item's, implemented as a pointer to heap-allocated GHashTable. vector_of_crash_data_t is a vector of crash_data_t's, implemented as a pointer to heap-allocated GPtrArray. Most operations have light wrappers around them to hide the nature of the containers. For example, to free vector_of_crash_data, you need to use free_vector_of_crash_data(ptr) instead of open-coding g_ptr_array_free. The wrapper is thin. The goal is not so much to hide the implementation, but more to make it easier to use the correct function. dbus (un)marshalling functions convert crash_item to three-element array of strings, in order to keep compatibility with abrt-gui (python). This can be changed later to use native representation. crash_data_t and vector_of_crash_data_t are represented in "natural" way, no funny stuff there. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* preparatory patch for future factoring out of headers for reportlibDenys Vlasenko2010-11-261-2/+2
| | | | | | Some files and functions are renamed, no logic changes. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* reorganize librariesDenys Vlasenko2010-11-261-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch does not change any code per se, it renames libABRTfoo -> libabrt_bar and moves a few functions around. After the patch, we are left with the following libs: libabrt - the stuff shared among most of abrt (like xmalloc, logging) libabrt_daemon - only daemon-related things are here (should probably be just moved into daemon - later) libabrt_dbus - daemon, cli and applet use this libabrt_web - abrt-action-foo where foo deals with network/web/ftp/... As a result, we have following reductions in dependent libs: /usr/libexec/abrt-hook-ccpp: linux-vdso.so.1 => () - libABRTUtils.so.0 => /usr/lib64/libABRTUtils.so.0 () + libabrt.so.0 => /usr/lib64/libabrt.so.0 () libglib-2.0.so.0 => /lib64/libglib-2.0.so.0 () - libdbus-1.so.3 => /lib64/libdbus-1.so.3 () - libpthread.so.0 => /lib64/libpthread.so.0 () - librt.so.1 => /lib64/librt.so.1 () - libdl.so.2 => /lib64/libdl.so.2 () /usr/libexec/abrt-action-upload: linux-vdso.so.1 => () libtar.so.1 => /usr/lib64/libtar.so.1 () libcurl.so.4 => /usr/lib64/libcurl.so.4 () - libABRTdUtils.so.0 => /usr/lib64/libABRTdUtils.so.0 () - libABRTUtils.so.0 => /usr/lib64/libABRTUtils.so.0 () + libabrt.so.0 => /usr/lib64/libabrt.so.0 () libglib-2.0.so.0 => /lib64/libglib-2.0.so.0 () - libdbus-1.so.3 => /lib64/libdbus-1.so.3 () Similar savings are in almost every abrt-action-foo. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* taint flag is decimal representation of a bitmaskNikola Pajkovsky2010-11-251-4/+68
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* rhbz 623142Nikola Pajkovsky2010-11-221-0/+9
| | | | | | | Save tainted flag from /proc/sys/kernel/tainted. When flag is set to 1 TAINED string is added to bugzilla summary. Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* fix locking bugzilla conf file when -d not specifiedNikola Pajkovsky2010-11-191-1/+0
| | | | | | | when -c option is specified and -d option is not bugzilla locks his own confing file Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* move files from lib/plugins to src/pluginsJiri Moskovcak2010-11-101-0/+908