summaryrefslogtreecommitdiffstats
path: root/src/cli
Commit message (Collapse)AuthorAgeFilesLines
* Ticket #167 abrt-cli doesn't generate backtraceNikola Pajkovsky2011-03-251-1/+1
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* Merge branch 'event'Nikola Pajkovsky2011-03-251-36/+66
|\ | | | | | | | | | | * event: validate input in cli and fix ask_for_missing_settings function parse and store 'allow-empty' value
| * validate input in cli and fix ask_for_missing_settings functionNikola Pajkovsky2011-03-221-36/+66
| | | | | | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* | abrt-cli: fix inverted logic check for -lDenys Vlasenko2011-03-181-1/+1
| | | | | | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* | Replace FILENAME_MESSAGE with FILENAME_REPORTED_TO.Denys Vlasenko2011-03-182-19/+2
|/ | | | | | | | | | | 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 'cli'Nikola Pajkovsky2011-03-171-150/+47
|\ | | | | | | | | | | * cli: cli: load ~/.abrt/events/foo.conf expose export/unexport_event_config
| * cli: load ~/.abrt/events/foo.confNikola Pajkovsky2011-03-171-150/+47
| | | | | | | | | | | | need to be implemented _rating_required_ and ask_for_missing_settings Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* | Remove deprecated ENABLE_SOCKET_OR_DBUS configuration optionKarel Klic2011-03-161-1/+0
|/
* CLI.cpp -> cli.cNikola Pajkovsky2011-03-113-1/+9
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* add -Wwrite-strings everywhereDenys Vlasenko2011-03-091-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* gui-wizard-gtk: add code to export/unexport config variables. UntestedDenys Vlasenko2011-03-071-8/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Revert "renamed libreport.so to libreport2.so"Karel Klic2011-03-031-1/+1
| | | | | This is not needed at the end. This reverts commit ef96cd431369b2cf0dd640a0bb66be4fb029b9c0.
* remove FILENAME_REPRODUCE field. FILENAME_COMMENT should be used insteadDenys Vlasenko2011-03-032-15/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* renamed libreport.so to libreport2.soJiri Moskovcak2011-03-031-1/+1
| | | | | - we have a conflict with old report library which contains file with the same name
* dump_dir API: rename dd_dir field to dd_dirnameDenys Vlasenko2011-02-281-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* change stealing semantics from copy to moveDenys Vlasenko2011-02-251-15/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* move steal_directory() into libreportJiri Moskovcak2011-02-171-41/+0
|
* get rid of unused CD_FLAG_SYSNikola Pajkovsky2011-02-162-17/+3
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>
* abrt-gtk: make Delete key actually delete the dump dirDenys Vlasenko2011-02-105-205/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* get rid of FILENAME_DESCRIPTION, rename "release" to "os_release"Denys Vlasenko2011-02-102-3/+3
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-cli: suppress errors when we try to read dump dirs which aren't oursDenys Vlasenko2011-02-071-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | $ abrt-cli -lf Can't create lock file '/var/spool/abrt/ccpp-1296609283-15129/.lock': Permission denied Can't access '/var/spool/abrt/ccpp-1296609283-15129': Permission denied Can't create lock file '/var/spool/abrt/ccpp-1294848465-26639/.lock': Permission denied Can't access '/var/spool/abrt/ccpp-1294848465-26639': Permission denied Can't create lock file '/var/spool/abrt/ccpp-1294848466-26657/.lock': Permission denied Can't access '/var/spool/abrt/ccpp-1294848466-26657': Permission denied ^^^^^^^^^^^^^^^^^^^^ these messages 0. Crash dump : /home/test/.abrt/spool/ccpp-1296609283-15129.868047 UID : 0 Package : coreutils-8.4-10.fc13 Executable : /bin/sleep Crash Time : Wed 02 Feb 2011 02:14:43 AM CET Crash Count: 1 Hostname : dhcp-25-227.brq.redhat.com 1. Crash dump : /home/test/.abrt/spool/ccpp-1296609283-15129 UID : 0 Package : coreutils-8.4-10.fc13 Executable : /bin/sleep Crash Time : Wed 02 Feb 2011 02:14:43 AM CET Crash Count: 1 Hostname : dhcp-25-227.brq.redhat.com Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-cli: delete half-copied dir on dir copy errorDenys Vlasenko2011-02-071-1/+14
| | | | | | Also, make -d DIR to try deletion itself before resorting to abrtd Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* cli: change user's abrt dir from $HOME/abrt to $HOME/.abrtDenys Vlasenko2011-02-031-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* abrt-cli -r DIR: copy non-writable DIR into $HOME/abrt/spoolDenys Vlasenko2011-02-011-8/+66
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* use run_state->children_count == 0 check for "event not conf'd" conditionDenys Vlasenko2011-02-011-8/+3
| | | | | | | | | | | | 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>
* abrt-cli: converted to process events locallyDenys Vlasenko2011-02-015-304/+370
| | | | | | Only -d DIR operation still goes through the daemon. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* preparatory changes for abrt-cli local processing changeDenys Vlasenko2011-01-274-152/+55
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* introduce and use new helper function list_free_with_freeDenys Vlasenko2011-01-202-3/+0
| | | | 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>
* small fixes to map_string_t conversion suggested by KarelDenys Vlasenko2010-12-131-8/+8
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* rename libabrt.so to libreport.soDenys Vlasenko2010-12-091-1/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* create report-libs-devel package; separate out report headersDenys Vlasenko2010-12-092-13/+11
| | | | 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 1Denys Vlasenko2010-12-082-16/+24
| | | | | | | | | | | | | 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>
* remove unused function parse_args; make a few functions extern "C"Denys Vlasenko2010-12-071-2/+2
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* This patch changes crash data to use C structures.Denys Vlasenko2010-12-064-89/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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: add -v to abrt-cli; remove unused func; make func staticDenys Vlasenko2010-12-032-21/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* 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>
* get_reporter_plugin_settings() returns GHashTableNikola Pajkovsky2010-12-013-23/+41
| | | | | | | | | | | | 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>
* preparatory patch for future factoring out of headers for reportlibDenys Vlasenko2010-11-261-1/+1
| | | | | | Some files and functions are renamed, no logic changes. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* reorganize librariesDenys Vlasenko2010-11-262-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* 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>
* remove sqlite DBDenys Vlasenko2010-11-233-106/+40
| | | | | | | | | | | | | | | | | | | | | | 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>
* move inc/ and lib/ to src/. No code changesDenys Vlasenko2010-11-151-4/+4
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* Move some functions from CrashWatcher to MiddleWareDenys Vlasenko2010-11-091-1/+1
| | | | | | They have nothing to do with CrashWatcher class. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* make #includes simplerDenys Vlasenko2010-11-051-3/+0
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* introduce and use xmalloc_fgets/fgetlineDenys Vlasenko2010-10-221-3/+2
| | | | | | | This fixes problems of having long lines truncated - and we do have very long lines sometimes - curl errors with HTML, list of debuginfos etc. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* s/CD_DUPHASH/FILENAME_DUPHASH/gDenys Vlasenko2010-10-111-2/+2
| | | | | | | This is needed to make duplicate hash visible for separate programs looking at crash dumps. Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* trivial: remove unused option_index'sDenys Vlasenko2010-10-081-2/+1
| | | | Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
* report.cpp: split() uses GListNikola Pajkovsky2010-10-062-11/+18
| | | | Signed-off-by: Nikola Pajkovsky <npajkovs@redhat.com>