From 16b7710b6a8548ca9ab58ca94fce60dcc92c1ba2 Mon Sep 17 00:00:00 2001 From: Karel Klic Date: Mon, 8 Feb 2010 20:36:19 +0100 Subject: Add component owner to the output --- src/Backtrace/abrt-bz-dupchecker | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/src/Backtrace/abrt-bz-dupchecker b/src/Backtrace/abrt-bz-dupchecker index cbdafc5..cb11bf1 100755 --- a/src/Backtrace/abrt-bz-dupchecker +++ b/src/Backtrace/abrt-bz-dupchecker @@ -25,6 +25,8 @@ import sys import os.path import subprocess import cPickle +import urllib +import json parser = OptionParser(version="%prog 1.0") parser.add_option("-u", "--user", dest="user", @@ -87,6 +89,7 @@ for buginfo in buginfos: if ids.has_key(buginfo.bug_id): continue + ids[buginfo.bug_id] = True if not buginfo.bug_status in ["NEW", "ASSIGNED", "MODIFIED", "VERIFIED"]: @@ -177,9 +180,17 @@ print "------------------------------" for backtrace, components in database.items(): for component, bugitems in components.items(): if len(bugitems) > 1: + # Get the component owner + component_info = json.load(urllib.urlopen("https://admin.fedoraproject.org/pkgdb/packages/name/{0}?tg_format=json".format(buginfo.component))) + component_packages = component_info['packageListings'] + component_f12 = filter(lambda x:x["collection"]["version"]=="12", component_packages) + owner = "Failed to get component owner" + if len(component_f12) == 1: + owner = component_f12["owner"] + if options.wiki: print "----" - print "* component: '''{0}'''".format(component) + print "* component: '''{0}''' ({1})".format(component, owner) print "* duplicates: {0}".format( reduce(lambda x,y: x+", "+y, map(lambda x: "#[https://bugzilla.redhat.com/show_bug.cgi?id={0} {0}] ({1} comments)".format(x['id'],x['comments']), @@ -188,7 +199,7 @@ for backtrace, components in database.items(): for line in backtrace.replace("Thread\n", "").splitlines(): print "*# {0}".format(line) else: - print "Component: {0}".format(component) + print "Component: {0} ({1})".format(component, owner) print "Duplicates: {0}".format( reduce(lambda x,y: x+", "+y, map(lambda x: "{0} ({1})".format(x['id'],x['comments']), -- cgit From f4d20785a4d6571225146fccc5367a4f032a8349 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Mon, 8 Feb 2010 21:38:29 +0100 Subject: remove FILENAME_UUID Signed-off-by: Denys Vlasenko --- inc/CrashTypes.h | 3 --- lib/Utils/make_descr.cpp | 1 - 2 files changed, 4 deletions(-) diff --git a/inc/CrashTypes.h b/inc/CrashTypes.h index b14413d..af106cf 100644 --- a/inc/CrashTypes.h +++ b/inc/CrashTypes.h @@ -26,9 +26,6 @@ #define FILENAME_KERNEL "kernel" #define FILENAME_TIME "time" #define FILENAME_UID "uid" -// uuid _file_ exists for Python analyzer only - remove, follow ccpp's example? -// (ccpp keeps uuid in DB) -#define FILENAME_UUID "uuid" #define FILENAME_PACKAGE "package" #define FILENAME_COMPONENT "component" #define FILENAME_DESCRIPTION "description" diff --git a/lib/Utils/make_descr.cpp b/lib/Utils/make_descr.cpp index c2821a5..6a1deda 100644 --- a/lib/Utils/make_descr.cpp +++ b/lib/Utils/make_descr.cpp @@ -71,7 +71,6 @@ static void add_content(bool &was_multiline, string& description, const char *he static const char *const blacklisted_items_bz[] = { FILENAME_TIME , FILENAME_UID , - FILENAME_UUID , FILENAME_ANALYZER , FILENAME_COREDUMP , FILENAME_DESCRIPTION, /* package description - basically useless */ -- cgit From 41ccc40114df0611992066e9c5d9d6322975ea17 Mon Sep 17 00:00:00 2001 From: Jiri Moskovcak Date: Mon, 8 Feb 2010 21:38:42 +0100 Subject: updated HOW_TO_TEST --- doc/HOW_TO_TEST | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/doc/HOW_TO_TEST b/doc/HOW_TO_TEST index d11c6d8..8e63c36 100644 --- a/doc/HOW_TO_TEST +++ b/doc/HOW_TO_TEST @@ -4,7 +4,9 @@ I. Build and install =================== a) build the packages as scratch in koji b) download and install (the best is to create a local repo and try to install it using yum) -c) start the daemon and check logs for any sign of a problem +c) create a repository and try to update from previous packages +d) start the daemon and check logs for any sign of a problem +e) try to update from previous Fedora version using created repository II. Crash detection ===================== @@ -29,4 +31,5 @@ III. Reporting - crash should be reported after filling the right credentials - login+pass won't be saved c) try to report every supported crash abrt detects C/C++, oops, python script - d) repeat this with cli + d) try to report some crash with bad backtrace - this should fail + e) repeat this with cli -- cgit From fb406ac2bd88b6c02d8bc8a814cc3095b9bd4868 Mon Sep 17 00:00:00 2001 From: Jiri Moskovcak Date: Mon, 8 Feb 2010 22:22:49 +0100 Subject: GUI: respect system settings for toolbars rhbz#552161 --- src/Gui/ccgui.glade | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/Gui/ccgui.glade b/src/Gui/ccgui.glade index 237f23a..652e6ff 100644 --- a/src/Gui/ccgui.glade +++ b/src/Gui/ccgui.glade @@ -208,7 +208,6 @@ Patrick Connelly <pcon@fedoraproject.org> True - both True @@ -225,7 +224,6 @@ Patrick Connelly <pcon@fedoraproject.org> True - False Report Report gtk-go-up -- cgit From c4425b1a6f3be01c0e7b8d58ebfd2b2df836ed0a Mon Sep 17 00:00:00 2001 From: Jiri Moskovcak Date: Tue, 9 Feb 2010 00:14:14 +0100 Subject: GUI: fixed exception in plugin settings dialog rhbz#560851 --- src/Gui/PluginsSettingsDialog.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Gui/PluginsSettingsDialog.py b/src/Gui/PluginsSettingsDialog.py index 0ba390d..d6129b1 100644 --- a/src/Gui/PluginsSettingsDialog.py +++ b/src/Gui/PluginsSettingsDialog.py @@ -141,7 +141,7 @@ class PluginsSettingsDialog: def on_bConfigurePlugin_clicked(self, button, pluginview): pluginsListStore, path = pluginview.get_selection().get_selected_rows() if not path: - self.builder.get_object("lDescription").set_label(_("Can't get plugin description")) + gui_info_dialog(_("Please select a plugin from the list to edit it's options."), self.window) return # this should work until we keep the row object in the last position pluginfo = pluginsListStore.get_value(pluginsListStore.get_iter(path[0]), pluginsListStore.get_n_columns()-1) -- cgit From 7cb783dabf6a55780d77263272e38f684ba3307c Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Tue, 9 Feb 2010 13:21:12 +0100 Subject: abrt.init: add commented-out http[s]_proxy=xxx assignments Signed-off-by: Denys Vlasenko --- abrt.init | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/abrt.init b/abrt.init index 6256767..c286c64 100644 --- a/abrt.init +++ b/abrt.init @@ -11,7 +11,7 @@ # Default-Stop: 0 1 2 6 # Default-Start: 3 5 # Short-Description: start and stop abrt daemon -# Description: Listen and dispatch crash events +# Description: Listen to and dispatch crash events ### END INIT INFO # Source function library. @@ -20,13 +20,19 @@ ABRT_BIN="/usr/sbin/abrtd" LOCK="/var/lock/subsys/abrtd" RETVAL=0 +# +# Set these variables if you are behind proxy +# +#export http_proxy= +#export https_proxy= + # # See how we were called. # check() { # Check that we're a privileged user - [ `id -u` = 0 ] || exit 4 + [ "`id -u`" = 0 ] || exit 4 # Check if abrt is executable test -x $ABRT_BIN || exit 5 -- cgit