summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* removal of C++isms from libabrt, part 2Denys Vlasenko2010-12-088-76/+24
| | | | | | 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-0829-485/+347
| | | | | | | | | | | | | 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>
* move libabrt.so to separate package report-libsJiri Moskovcak2010-12-071-1/+17
|
* remove unused function parse_args; make a few functions extern "C"Denys Vlasenko2010-12-075-71/+9
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* This patch changes crash data to use C structures.Denys Vlasenko2010-12-0626-458/+749
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* l10n: Updates to Czech (cs) translationppisar2010-12-061-146/+159
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Dutch (Flemish) (nl) translationwarrink2010-12-041-70/+72
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Polish (pl) translationraven2010-12-041-69/+71
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Ukrainian (uk) translationyurchor2010-12-041-68/+70
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Added Low German translationncfiedler2010-12-031-2/+2
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Added Low German translationncfiedler2010-12-031-0/+1581
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* preparatory patch: add -v to abrt-cli; remove unused func; make func staticDenys Vlasenko2010-12-035-37/+16
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* GUI: optimized the log window (possibly fixes rhbz#596592)Jiri Moskovcak2010-12-022-37/+50
| | | | | | - we used to change the whole textbuffer and redraw the whole textview, now we just add the last line - possibly fixes *gtk_text_mark_get_buffer bugs
* GHashTable: must use g_hash_table_replace instead of _insert to not leak keyDenys Vlasenko2010-12-011-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* factor out headers so that "report lib" API is separated out of abrt APIDenys Vlasenko2010-12-0114-67/+284
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* get_reporter_plugin_settings() returns GHashTableNikola Pajkovsky2010-12-014-23/+70
| | | | | | | | | | | | static void get_reporter_plugin_settings(const vector_string_t& reporters, map_map_string_t &settings) a new interface is static GHashTable *get_reporter_plugin_settings(const vector_string_t& reporters) Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* l10n: Updates to Ukrainian (uk) translationyurchor2010-12-011-63/+64
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Polish (pl) translationraven2010-11-301-28/+26
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Dutch (Flemish) (nl) translationwarrink2010-11-301-31/+26
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* updated po filesJiri Moskovcak2010-11-2942-6783/+11396
|
* new debuginfo install script rewritten in pythonJiri Moskovcak2010-11-296-426/+480
| | | | | | | | - using python alows us to use the yum API, so we can read the progress, file sizes, requires disk space, etc.. and seems to be faster the using yum --whatprovides + yumdownloader - it's easier to translate - we can drop dependency on yum-utils
* fix locking kerneloops conf file when -d not specifiedNikola Pajkovsky2010-11-291-1/+0
| | | | | | | when -c option is specified and -d option is not, keneloops locks his own confing file Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* add a new binary files into .gitignoreNikola Pajkovsky2010-11-291-0/+13
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* l10n: Updates to Russian (ru) translationshnurapet2010-11-261-416/+290
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* preparatory patch for future factoring out of headers for reportlibDenys Vlasenko2010-11-2625-43/+42
| | | | | | Some files and functions are renamed, no logic changes. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-action-rhtsupport: fix -c CONFFILE erroneously setting DUMPDIR tooDenys Vlasenko2010-11-261-1/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* correct a comment about libraries' usersDenys Vlasenko2010-11-261-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* reorganize librariesDenys Vlasenko2010-11-2624-268/+250
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* GUI: added a button to view a log when in verbose modeJiri Moskovcak2010-11-251-0/+30
|
* GUI: at least show warn when the rating is lowJiri Moskovcak2010-11-251-0/+8
| | | | | | - we lost the ability to query the reporters if they require rating during the plugins->separate process transition, so we can't disable the reporting, so at least show a warning about it...
* fixed build against libnotify 0.7Jiri Moskovcak2010-11-251-0/+7
|
* abrtd: suppress spurious warning about missing coredump and inform_allDenys Vlasenko2010-11-253-9/+14
| | | | | | Also plugs two memory leaks Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* taint flag is decimal representation of a bitmaskNikola Pajkovsky2010-11-252-14/+76
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* abrt-action-install-debuginfo: removed generation of MISSING: stringDenys Vlasenko2010-11-241-5/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-cli: dont die if message element is missing in the dump dirDenys Vlasenko2010-11-241-2/+6
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* l10n: Updates to Dutch (Flemish) (nl) translationwarrink2010-11-241-59/+55
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Polish (pl) translationraven2010-11-231-54/+44
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* GUI: handle crashdir parameter in crash_cbJiri Moskovcak2010-11-231-1/+1
| | | | | | - crashdir in now in Crash signal, so we need to modify crash_cb to know about it, otherwise it ends up with exception and gui is not refreshed when crash happens while the gui is open
* remove sqlite DBDenys Vlasenko2010-11-2344-1682/+369
| | | | | | | | | | | | | | | | | | | | | | This change removes sqlite database. Database was used to find dump dirs by [UID:]UUID. This patch uses more natural way: dump dirs are addressed by their directory names. DB was also used to produce a list of dump dirs. Now it is done by iterating over the /var/spool/abrt directory. And finally, DB was also used to find duplicate UUIDs. Now it is done by iterating over the /var/spool/abrt directory. Crash count, "inform all" and reporting result message are moved from DB field to a file in dump dir. "Reported" DB field is deleted - if message != "", then this dump was reported. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* l10n: Updates to Dutch (Flemish) (nl) translationwarrink2010-11-231-23/+29
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* l10n: Updates to Polish (pl) translationraven2010-11-231-114/+29
| | | | Transmitted-via: Transifex (translate.fedoraproject.org)
* added po/notexist to .gitignoreJiri Moskovcak2010-11-221-0/+1
|
* updated po files and POTFILES.in rhbz#654618Jiri Moskovcak2010-11-2244-19974/+22864
|
* rhbz 623142Nikola Pajkovsky2010-11-223-0/+28
| | | | | | | 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>
* Merge branch 'remove-polkit'Nikola Pajkovsky2010-11-2214-336/+1
|\ | | | | | | | | | | * remove-polkit: remove PLUGIN_DYNAMIC_LOAD_UNLOAD remove polkit
| * remove PLUGIN_DYNAMIC_LOAD_UNLOADNikola Pajkovsky2010-11-195-129/+0
| | | | | | | | | | | | it contains polkit Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
| * remove polkitNikola Pajkovsky2010-11-1910-207/+1
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* | preraratory patch for database removalDenys Vlasenko2010-11-224-19/+28
| | | | | | | | | | | | | | I splitted bits from a large "database removal" patch which are simple and non-contentiouns. They are in this change. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* | Improve some comments. No code changesDenys Vlasenko2010-11-222-7/+13
| | | | | | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* | GUI: made howto mandatoryJiri Moskovcak2010-11-221-4/+50
| | | | | | | | - patch taken from 1.1.x branch, so pushing it without the review