On Thu, 2010-03-04 at 17:39 +0100, Nikola Pajkovsky wrote:
Should I add more information?
---
>From e236245ad3fd855a9caf2c6ff5f08d864f4d928a Mon Sep 17 00:00:00 2001
From: Nikola Pajkovsky <npajkovs(a)redhat.com>
Date: Thu, 4 Mar 2010 17:17:49 +0100
Subject: [PATCH] add arch, package and release in comment
Signed-off-by: Nikola Pajkovsky <npajkovs(a)redhat.com>
---
lib/Plugins/Bugzilla.cpp | 11 ++++++++++-
1 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/lib/Plugins/Bugzilla.cpp b/lib/Plugins/Bugzilla.cpp
index 6ec1f42..39ccafb 100644
--- a/lib/Plugins/Bugzilla.cpp
+++ b/lib/Plugins/Bugzilla.cpp
@@ -677,7 +677,16 @@ std::string CReporterBugzilla::Report(const
map_crash_data_t& pCrashData,
std::string description =
make_description_reproduce_comment(pCrashData);
if (!description.empty())
{
- VERB3 log("Add new comment into bug(%d)", (int)bug_id);
+ const char* package =
get_crash_data_item_content_or_NULL(pCrashData, FILENAME_PACKAGE);
Indentation is wrong.
+ const char* release =
get_crash_data_item_content_or_NULL(pCrashData, FILENAME_RELEASE);
+ const char* arch =
get_crash_data_item_content_or_NULL(pCrashData, FILENAME_ARCHITECTURE);
+
+ description = ssprintf("Package: %s\n"
+ "Architecture: %s\n"
+ "Release: %s\n"
Maybe name it "OS Release" (as opposed to package's release).
+ "%s", package, arch, release,
description.c_str()
+ );
+
update_client(_("Add new comment into bug(%d)"), (int)bug_id);
if (bz_server.add_comment(bug_id, description.c_str()) == -1)
{
--
vda