[PATCH] move logger reporting to a separate program (abrt-action-print)

[Denys Vlasenko]

# abrt-action-print --help
Usage: abrt-action-print [v] -d DIR

Print information about the crash to standard output

    -v, --verbose         be verbose
    -d DIR                Crash dump directory

As you can see, it simply writes crash dump information
to stdout, not to a file.

Logger plugin spawns a child abrt-action-print -d DIR
and pipes its output to log file.

Run tested.

Please review.

-- 
vda



diff -x '*.po' -d -urpN abrt.7/abrt.spec abrt.8/abrt.spec
--- abrt.7/abrt.spec	2010-10-18 17:49:43.560968358 +0200
+++ abrt.8/abrt.spec	2010-10-19 15:49:53.829565690 +0200
@@ -356,6 +356,7 @@ fi
 %{_sbindir}/abrt-action-save-package-data
 %{_bindir}/abrt-action-bugzilla
 %{_bindir}/abrt-action-rhtsupport
+%{_bindir}/abrt-action-print
 %{_bindir}/abrt-action-install-debuginfo
 %{_bindir}/%{name}-handle-upload
 %{_bindir}/%{name}-backtrace
diff -x '*.po' -d -urpN abrt.7/lib/plugins/Logger.cpp
abrt.8/lib/plugins/Logger.cpp
--- abrt.7/lib/plugins/Logger.cpp	2010-10-11 19:29:27.000000000 +0200
+++ abrt.8/lib/plugins/Logger.cpp	2010-10-19 15:59:27.182316435 +0200
@@ -1,5 +1,5 @@
 /*
-    Logger.cpp - it simple writes report to specific file
+    Logger.cpp - it simply writes report to a specified file
 
     Copyright (C) 2009  Zdenek Prikryl (zprikryl(a)redhat.com)
     Copyright (C) 2009  RedHat inc.
@@ -19,60 +19,107 @@
     51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */
 #include "abrtlib.h"
-#include "Logger.h"
 #include "comm_layer_inner.h"
 #include "abrt_exception.h"
+#include "Logger.h"
+
+using namespace std;
 
 CLogger::CLogger()
 {
-    m_log_path = xstrdup("/var/log/abrt.log");
-    m_append_logs = true;
+    m_pSettings["LogPath"] = "/var/log/abrt.log";
+    m_pSettings["AppendLogs"] = "yes";
 }
 
 CLogger::~CLogger()
 {
-    free(m_log_path);
 }
 
 void CLogger::SetSettings(const map_plugin_settings_t& pSettings)
 {
+    /* Can't simply do this:
+
     m_pSettings = pSettings;
 
-    map_plugin_settings_t::const_iterator end = pSettings.end();
-    map_plugin_settings_t::const_iterator it;
-    it = pSettings.find("LogPath");
-    if (it != end)
+     * - it will erase keys which aren't present in pSettings.
+     * Example: if Bugzilla.conf doesn't have "Login = foo",
+     * then there's no pSettings["Login"] and m_pSettings = pSettings
+     * will nuke default m_pSettings["Login"] = "",
+     * making GUI think that we have no "Login" key at all
+     * and thus never overriding it - even if it *has* an override!
+     */
+
+    map_plugin_settings_t::iterator it = m_pSettings.begin();
+    while (it != m_pSettings.end())
     {
-        free(m_log_path);
-        m_log_path = xstrdup(it->second.c_str());
+        map_plugin_settings_t::const_iterator override = pSettings.find(it->first);
+        if (override != pSettings.end())
+        {
+            VERB3 log(" logger settings[%s]='%s'",
it->first.c_str(), it->second.c_str());
+            it->second = override->second;
+        }
+        it++;
     }
-    it = pSettings.find("AppendLogs");
-    if (it != end)
-        m_append_logs = string_to_bool(it->second.c_str());
 }
 
-std::string CLogger::Report(const map_crash_data_t& pCrashData,
+string CLogger::Report(const map_crash_data_t& crash_data,
                 const map_plugin_settings_t& pSettings,
                 const char *pArgs)
 {
-    char *dsc = make_description_logger(pCrashData);
-    char *full_dsc = xasprintf("%s\n\n\n", dsc);
-    free(dsc);
+    const char *log_path = "/var/log/abrt.log";
+    bool append_logs = true;
+
+    map_plugin_settings_t::const_iterator end = pSettings.end();
+    map_plugin_settings_t::const_iterator it;
+    it = pSettings.find("LogPath");
+    if (it != end)
+        log_path = it->second.c_str();
+    it = pSettings.find("AppendLogs");
+    if (it != end)
+        append_logs = string_to_bool(it->second.c_str());
 
     /* open, not fopen - want to set mode if we create the file, not just open */
-    const char *fname = m_log_path;
-    int fd = open(fname, m_append_logs ? O_WRONLY|O_CREAT|O_APPEND :
O_WRONLY|O_CREAT|O_TRUNC, 0600);
+    int fd = open(log_path, append_logs ? O_WRONLY|O_CREAT|O_APPEND :
O_WRONLY|O_CREAT|O_TRUNC, 0600);
     if (fd < 0)
-        throw CABRTException(EXCEP_PLUGIN, "Can't open '%s'",
fname);
+        throw CABRTException(EXCEP_PLUGIN, "Can't open '%s'",
log_path);
 
-    update_client(_("Writing report to '%s'"), fname);
-    full_write_str(fd, full_dsc);
-    free(full_dsc);
+    update_client(_("Writing report to '%s'"), log_path);
 
+    /* abrt-action-print -d DIR NULL */
+    char *argv[4];
+    char **pp = argv;
+    *pp++ = (char*)"abrt-action-print";
+    *pp++ = (char*)"-d";
+    *pp++ = (char*)get_crash_data_item_content_or_NULL(crash_data, CD_DUMPDIR);
+    *pp = NULL;
+    int pipefds[2];
+    pid_t pid = fork_execv_on_steroids(EXECFLG_OUTPUT, // + EXECFLG_ERR2OUT,
+                argv,
+                pipefds,
+                /* unsetenv_vec: */ NULL,
+                /* dir: */ NULL,
+                /* uid(unused): */ 0
+    );
+
+    /* Consume log from stdout, write it to log file */
+    FILE *fp = fdopen(pipefds[0], "r");
+    if (!fp)
+        die_out_of_memory();
+    char buf[512];
+    while (fgets(buf, sizeof(buf), fp))
+    {
+	full_write_str(fd, buf);
+    }
+    fclose(fp); /* this also closes pipefds[0] */
+    /* wait for child to actually exit, and prevent leaving a zombie behind */
+    waitpid(pid, NULL, 0);
+
+    /* Add separating empty lines, then close log file */
+    full_write_str(fd, "\n\n\n");
     close(fd);
 
-    const char *format = m_append_logs ? _("The report was appended to %s") :
_("The report was stored to %s");
-    return ssprintf(format, m_log_path);
+    const char *format = append_logs ? _("The report was appended to %s") :
_("The report was stored to %s");
+    return ssprintf(format, log_path);
 }
 
 PLUGIN_INFO(REPORTER,
diff -x '*.po' -d -urpN abrt.7/lib/plugins/Logger.h abrt.8/lib/plugins/Logger.h
--- abrt.7/lib/plugins/Logger.h	2010-09-23 16:48:44.000000000 +0200
+++ abrt.8/lib/plugins/Logger.h	2010-10-19 16:20:13.408315878 +0200
@@ -27,9 +27,6 @@
 
 class CLogger : public CReporter
 {
-    private:
-        char *m_log_path;
-        bool m_append_logs;
     public:
         CLogger();
         ~CLogger();
diff -x '*.po' -d -urpN abrt.7/src/daemon/abrt-action-print.cpp
abrt.8/src/daemon/abrt-action-print.cpp
--- abrt.7/src/daemon/abrt-action-print.cpp	1970-01-01 01:00:00.000000000 +0100
+++ abrt.8/src/daemon/abrt-action-print.cpp	2010-10-19 16:07:55.279315726 +0200
@@ -0,0 +1,83 @@
+/*
+    Write crash dump to stdout in text form.
+
+    Copyright (C) 2009  Zdenek Prikryl (zprikryl(a)redhat.com)
+    Copyright (C) 2009  RedHat inc.
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License along
+    with this program; if not, write to the Free Software Foundation, Inc.,
+    51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+*/
+#include "abrtlib.h"
+#include "parse_options.h"
+#include "crash_types.h"
+#include "abrt_exception.h"
+#include "plugin.h" /* make_description_logger */
+
+#define PROGNAME "abrt-action-print"
+
+static void report_to_stdout(const char *dump_dir_name)
+{
+    struct dump_dir *dd = dd_opendir(dump_dir_name, /*flags:*/ 0);
+    if (!dd)
+    {
+        throw CABRTException(EXCEP_PLUGIN, _("Can't open '%s'"),
dump_dir_name);
+    }
+    map_crash_data_t pCrashData;
+    load_crash_data_from_debug_dump(dd, pCrashData);
+    dd_close(dd);
+
+    char *dsc = make_description_logger(pCrashData);
+    fputs(dsc, stdout);
+    free(dsc);
+}
+
+static const char *dump_dir_name = ".";
+
+int main(int argc, char **argv)
+{
+    char *env_verbose = getenv("ABRT_VERBOSE");
+    if (env_verbose)
+        g_verbose = atoi(env_verbose);
+
+    const char *program_usage = _(
+        PROGNAME" [v] -d DIR\n"
+        "\n"
+        "Print information about the crash to standard output");
+    enum {
+        OPT_v = 1 << 0,
+        OPT_d = 1 << 1,
+    };
+    /* Keep enum above and order of options below in sync! */
+    struct options program_options[] = {
+        OPT__VERBOSE(&g_verbose),
+        OPT_STRING('d', NULL, &dump_dir_name, "DIR", _("Crash
dump directory")),
+        OPT_END()
+    };
+
+    /*unsigned opts =*/ parse_opts(argc, argv, program_options, program_usage);
+
+    putenv(xasprintf("ABRT_VERBOSE=%u", g_verbose));
+
+    try
+    {
+        report_to_stdout(dump_dir_name);
+    }
+    catch (CABRTException& e)
+    {
+        log("%s", e.what());
+        return 1;
+    }
+
+    return 0;
+}
diff -x '*.po' -d -urpN abrt.7/src/daemon/Makefile.am
abrt.8/src/daemon/Makefile.am
--- abrt.7/src/daemon/Makefile.am	2010-10-18 18:18:06.370302316 +0200
+++ abrt.8/src/daemon/Makefile.am	2010-10-19 15:50:56.476565669 +0200
@@ -12,7 +12,8 @@ sbin_PROGRAMS = abrtd \
 
 bin_PROGRAMS = \
     abrt-action-bugzilla \
-    abrt-action-rhtsupport
+    abrt-action-rhtsupport \
+    abrt-action-print
 
 abrtd_SOURCES = \
     PluginManager.h PluginManager.cpp \
@@ -199,6 +200,26 @@ abrt_action_rhtsupport_LDADD = \
     ../../lib/utils/libABRTdUtils.la \
     ../../lib/utils/libABRTUtils.la
 
+abrt_action_print_SOURCES = \
+    abrt-action-print.cpp
+abrt_action_print_CPPFLAGS = \
+    -I$(srcdir)/../../inc \
+    -I$(srcdir)/../../lib/utils \
+    -DBIN_DIR=\"$(bindir)\" \
+    -DVAR_RUN=\"$(VAR_RUN)\" \
+    -DCONF_DIR=\"$(CONF_DIR)\" \
+    -DLOCALSTATEDIR='"$(localstatedir)"' \
+    -DDEBUG_DUMPS_DIR=\"$(DEBUG_DUMPS_DIR)\" \
+    -DDEBUG_INFO_DIR=\"$(DEBUG_INFO_DIR)\" \
+    -DPLUGINS_LIB_DIR=\"$(PLUGINS_LIB_DIR)\" \
+    -DPLUGINS_CONF_DIR=\"$(PLUGINS_CONF_DIR)\" \
+    $(GLIB_CFLAGS) \
+    -D_GNU_SOURCE \
+    -Wall -Werror
+abrt_action_print_LDADD = \
+    ../../lib/utils/libABRTdUtils.la \
+    ../../lib/utils/libABRTUtils.la
+
 dbusabrtconfdir = ${sysconfdir}/dbus-1/system.d/
 dist_dbusabrtconf_DATA = dbus-abrt.conf

[Jiri Moskovcak]

On 10/19/2010 04:24 PM, Denys Vlasenko wrote:
>    %{_bindir}/abrt-action-bugzilla
>    %{_bindir}/abrt-action-rhtsupport
>  +%{_bindir}/abrt-action-print 
- please don't add these executables to main package, it should be in 
the respective plugin's sub packages.

J.

[Denys Vlasenko]

On Tue, 2010-10-19 at 17:10 +0200, Jiri Moskovcak wrote:
>  On 10/19/2010 04:24 PM, Denys Vlasenko wrote:
>  >   %{_bindir}/abrt-action-bugzilla
>  >   %{_bindir}/abrt-action-rhtsupport
>  > +%{_bindir}/abrt-action-print
>  
>  - please don't add these executables to main package, it should be in 
>  the respective plugin's sub packages. 
Right... the following patch should do it (for all abrt-action-FOOs,
not only last ones):

-- 
vda



diff -x '*.po' -d -urpN abrt.2/abrt.spec abrt.3/abrt.spec
--- abrt.2/abrt.spec	2010-10-19 15:49:53.829565690 +0200
+++ abrt.3/abrt.spec	2010-10-20 17:21:00.507809244 +0200
@@ -69,7 +69,7 @@ Libraries for %{name}.
 %package devel
 Summary: Development libraries for %{name}
 Group: Development/Libraries
-Requires: %{name}-libs = %{version}-%{release}
+Requires: abrt-libs = %{version}-%{release}
 
 %description devel
 Development libraries and headers for %{name}.
@@ -201,9 +201,9 @@ uncaught exception in python programs.
 Summary: %{name}'s command line interface
 Group: User Interface/Desktops
 Requires: %{name} = %{version}-%{release}
-Requires: %{name}-addon-kerneloops
-Requires: %{name}-addon-ccpp, %{name}-addon-python
-Requires: %{name}-plugin-bugzilla, %{name}-plugin-logger, %{name}-plugin-runapp
+Requires: abrt-addon-kerneloops
+Requires: abrt-addon-ccpp, abrt-addon-python
+Requires: abrt-plugin-bugzilla, abrt-plugin-logger, abrt-plugin-runapp
 
 %description cli
 This package contains simple command line client for controlling abrt daemon over
@@ -218,13 +218,13 @@ Group: User Interface/Desktops
 # Installing abrt-desktop should result in the abrt which works without
 # any tweaking in abrt.conf (IOW: all plugins mentioned there must be installed)
 Requires: %{name} = %{version}-%{release}
-Requires: %{name}-addon-kerneloops
-Requires: %{name}-addon-ccpp, %{name}-addon-python
+Requires: abrt-addon-kerneloops
+Requires: abrt-addon-ccpp, abrt-addon-python
 # Default config of addon-ccpp requires gdb
 Requires: gdb >= 7.0-3
-Requires: %{name}-gui
-Requires: %{name}-plugin-logger, %{name}-plugin-bugzilla, %{name}-plugin-runapp
-#Requires: %{name}-plugin-firefox
+Requires: abrt-gui
+Requires: abrt-plugin-logger, abrt-plugin-bugzilla, abrt-plugin-runapp
+#Requires: abrt-plugin-firefox
 Obsoletes: bug-buddy
 Provides: bug-buddy
 
@@ -252,12 +252,12 @@ make install DESTDIR=$RPM_BUILD_ROOT man
 find $RPM_BUILD_ROOT -name '*.la' -or -name '*.a' | xargs rm -f
 mkdir -p ${RPM_BUILD_ROOT}/%{_initrddir}
 install -m 755 %SOURCE1 ${RPM_BUILD_ROOT}/%{_initrddir}/abrtd
-# /var/cache/%{name} is to be removed in 1.3.x timeframe
-mkdir -p $RPM_BUILD_ROOT/var/cache/%{name}
-mkdir -p $RPM_BUILD_ROOT/var/cache/%{name}-di
-mkdir -p $RPM_BUILD_ROOT/var/run/%{name}
-mkdir -p $RPM_BUILD_ROOT/var/spool/%{name}
-mkdir -p $RPM_BUILD_ROOT/var/spool/%{name}-upload
+# /var/cache/abrt is to be removed in 1.3.x timeframe
+mkdir -p $RPM_BUILD_ROOT/var/cache/abrt
+mkdir -p $RPM_BUILD_ROOT/var/cache/abrt-di
+mkdir -p $RPM_BUILD_ROOT/var/run/abrt
+mkdir -p $RPM_BUILD_ROOT/var/spool/abrt
+mkdir -p $RPM_BUILD_ROOT/var/spool/abrt-upload
 
 desktop-file-install \
         --dir ${RPM_BUILD_ROOT}%{_datadir}/applications \
@@ -267,7 +267,7 @@ desktop-file-install \
 
 desktop-file-install \
         --dir ${RPM_BUILD_ROOT}%{_sysconfdir}/xdg/autostart \
-        src/applet/%{name}-applet.desktop
+        src/applet/abrt-applet.desktop
 
 %clean
 rm -rf $RPM_BUILD_ROOT
@@ -349,39 +349,29 @@ fi
 %endif
 %{_sbindir}/abrtd
 %{_sbindir}/abrt-server
-%{_sbindir}/abrt-action-analyze-c
-%{_sbindir}/abrt-action-analyze-python
-%{_sbindir}/abrt-action-analyze-oops
-%{_sbindir}/abrt-action-generate-backtrace
-%{_sbindir}/abrt-action-save-package-data
-%{_bindir}/abrt-action-bugzilla
-%{_bindir}/abrt-action-rhtsupport
-%{_bindir}/abrt-action-print
-%{_bindir}/abrt-action-install-debuginfo
-%{_bindir}/%{name}-handle-upload
-%{_bindir}/%{name}-backtrace
-%config(noreplace) %{_sysconfdir}/%{name}/%{name}.conf
+%{_bindir}/abrt-handle-upload
+%config(noreplace) %{_sysconfdir}/%{name}/abrt.conf
 %config(noreplace) %{_sysconfdir}/%{name}/gpg_keys
-%config(noreplace) %{_sysconfdir}/dbus-1/system.d/dbus-%{name}.conf
-%{_initrddir}/%{name}d
-# /var/cache/%{name} is to be removed in 1.3.x timeframe
-%dir %attr(0755, abrt, abrt) %{_localstatedir}/cache/%{name}
-%dir %attr(0755, abrt, abrt) %{_localstatedir}/spool/%{name}
-%dir %attr(0700, abrt, abrt) %{_localstatedir}/spool/%{name}-upload
-%dir /var/run/%{name}
+%config(noreplace) %{_sysconfdir}/dbus-1/system.d/dbus-abrt.conf
+%{_initrddir}/abrtd
+# /var/cache/abrt is to be removed in 1.3.x timeframe
+%dir %attr(0755, abrt, abrt) %{_localstatedir}/cache/abrt
+%dir %attr(0755, abrt, abrt) %{_localstatedir}/spool/abrt
+%dir %attr(0700, abrt, abrt) %{_localstatedir}/spool/abrt-upload
+%dir /var/run/abrt
 %dir %{_sysconfdir}/%{name}
 %dir %{_sysconfdir}/%{name}/plugins
 %dir %{_libdir}/%{name}
-%{_mandir}/man1/%{name}-backtrace.1.gz
+%{_mandir}/man1/abrt-backtrace.1.gz
 %{_mandir}/man8/abrtd.8.gz
-%{_mandir}/man5/%{name}.conf.5.gz
+%{_mandir}/man5/abrt.conf.5.gz
 #%{_mandir}/man5/pyhook.conf.5.gz
-%{_mandir}/man7/%{name}-plugins.7.gz
+%{_mandir}/man7/abrt-plugins.7.gz
 %{_datadir}/polkit-1/actions/org.fedoraproject.abrt.policy
 %{_datadir}/dbus-1/system-services/com.redhat.abrt.service
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/SQLite3.conf
 %{_libdir}/%{name}/libSQLite3.so*
-%{_mandir}/man7/%{name}-SQLite3.7.gz
+%{_mandir}/man7/abrt-SQLite3.7.gz
 
 %files libs
 %defattr(-,root,root,-)
@@ -396,7 +386,7 @@ fi
 
 %files gui
 %defattr(-,root,root,-)
-%{_bindir}/%{name}-gui
+%{_bindir}/abrt-gui
 %dir %{_datadir}/%{name}
 # all glade, gtkbuilder and py files for gui
 %{_datadir}/%{name}/*.py*
@@ -405,15 +395,20 @@ fi
 %{_datadir}/icons/hicolor/*/apps/*
 %{_datadir}/icons/hicolor/*/status/*
 %{_datadir}/%{name}/icons/hicolor/*/status/*
-%{_bindir}/%{name}-applet
-%{_sysconfdir}/xdg/autostart/%{name}-applet.desktop
+%{_bindir}/abrt-applet
+%{_sysconfdir}/xdg/autostart/abrt-applet.desktop
 
 %files addon-ccpp
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/CCpp.conf
-%dir %{_localstatedir}/cache/%{name}-di
+%dir %{_localstatedir}/cache/abrt-di
 %{_libdir}/%{name}/libCCpp.so*
 %{_libexecdir}/abrt-hook-ccpp
+%{_bindir}/abrt-backtrace
+%{_sbindir}/abrt-action-analyze-c
+%{_bindir}/abrt-action-install-debuginfo
+%{_sbindir}/abrt-action-generate-backtrace
+%{_sbindir}/abrt-action-save-package-data
 
 %files addon-kerneloops
 %defattr(-,root,root,-)
@@ -421,67 +416,71 @@ fi
 %{_bindir}/dumpoops
 %{_libdir}/%{name}/libKerneloops.so*
 %{_libdir}/%{name}/libKerneloopsScanner.so*
-%{_mandir}/man7/%{name}-KerneloopsScanner.7.gz
+%{_mandir}/man7/abrt-KerneloopsScanner.7.gz
 %{_libdir}/%{name}/libKerneloopsReporter.so*
 %{_libdir}/%{name}/KerneloopsReporter.glade
-%{_mandir}/man7/%{name}-KerneloopsReporter.7.gz
+%{_mandir}/man7/abrt-KerneloopsReporter.7.gz
+%{_sbindir}/abrt-action-analyze-oops
 
 %files plugin-logger
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/Logger.conf
 %{_libdir}/%{name}/libLogger.so*
 %{_libdir}/%{name}/Logger.glade
-%{_mandir}/man7/%{name}-Logger.7.gz
+%{_mandir}/man7/abrt-Logger.7.gz
+%{_bindir}/abrt-action-print
 
 %files plugin-mailx
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/Mailx.conf
 %{_libdir}/%{name}/libMailx.so*
 %{_libdir}/%{name}/Mailx.glade
-%{_mandir}/man7/%{name}-Mailx.7.gz
+%{_mandir}/man7/abrt-Mailx.7.gz
 
 %files plugin-runapp
 %defattr(-,root,root,-)
 %{_libdir}/%{name}/libRunApp.so*
-%{_mandir}/man7/%{name}-RunApp.7.gz
+%{_mandir}/man7/abrt-RunApp.7.gz
 
 %files plugin-sosreport
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/SOSreport.conf
 %{_libdir}/%{name}/libSOSreport.so*
 
-
 %files plugin-bugzilla
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/Bugzilla.conf
 %{_libdir}/%{name}/libBugzilla.so*
 %{_libdir}/%{name}/Bugzilla.glade
-%{_mandir}/man7/%{name}-Bugzilla.7.gz
+%{_mandir}/man7/abrt-Bugzilla.7.gz
+%{_bindir}/abrt-action-bugzilla
 
 %files plugin-rhtsupport
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/RHTSupport.conf
 %{_libdir}/%{name}/libRHTSupport.so*
 %{_libdir}/%{name}/RHTSupport.glade
-#%{_mandir}/man7/%{name}-RHTSupport.7.gz
+#%{_mandir}/man7/abrt-RHTSupport.7.gz
+%{_bindir}/abrt-action-rhtsupport
 
 %files plugin-reportuploader
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/ReportUploader.conf
 %{_libdir}/%{name}/libReportUploader.so*
 %{_libdir}/%{name}/ReportUploader.glade
-%{_mandir}/man7/%{name}-ReportUploader.7.gz
+%{_mandir}/man7/abrt-ReportUploader.7.gz
 
 %files plugin-filetransfer
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/FileTransfer.conf
 %{_libdir}/%{name}/libFileTransfer.so*
-%{_mandir}/man7/%{name}-FileTransfer.7.gz
+%{_mandir}/man7/abrt-FileTransfer.7.gz
 
 %files addon-python
 %defattr(-,root,root,-)
 %config(noreplace) %{_sysconfdir}/%{name}/plugins/Python.conf
 %{_libdir}/%{name}/libPython.so*
+%{_sbindir}/abrt-action-analyze-python
 %{python_site}/*.py*
 %{python_site}/abrt.pth