[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Crash handler improvement
From:       Frank Reininghaus <frank78ac () googlemail ! com>
Date:       2008-08-02 10:04:50
Message-ID: 200808021204.50658.frank78ac () googlemail ! com
[Download RAW message or body]

Hi everyone,

some time ago, I read Aaron's blog entry 
http://aseigo.blogspot.com/2008/06/if-distros-wanted-to-be-useful.html
where he says that distros should ship beta packages with debugging info by 
default because this would remove the need to link to the "How to create 
useful crash reports" page endlessly in bug reports.

I agree, but there's something we can do without having to rely on distros: 
Put the link into the crash handler dialog. I've attached a patch that does 
it. I've chosen a rather conservative formulation instead of saying "You MUST 
install debugging packages!" because I think we shouldn't make users who 
cannot do it for whatever reason (no root access, not enough bandwidth to 
download hundreds of megabytes) refrain from filing bug reports. A bug 
report without backtrace, but with good instructions that enable us to 
reproduce it is still better than no bug report at all.

I also suggest to make the "Useful details include..." phrase a bit more 
verbose, see my change in kdebase/runtime/drkonqi/presets/enduserrc. Of 
course, I only changed the English version ;-)

It's just a suggestion and far from perfect (I should make sure the dialog is 
resized properly, right now it gets a bit messed up when I enable "Show 
details"), but I think something should be done to make the life of 
developers and bug triagers easier.

Cheers
Frank

["patch-crash-handler.diff" (text/x-diff)]

Index: runtime/drkonqi/presets/enduserrc
===================================================================
--- runtime/drkonqi/presets/enduserrc	(revision 840918)
+++ runtime/drkonqi/presets/enduserrc	(working copy)
@@ -199,7 +199,7 @@
 
 [WhatToDoHint]
 Enable=true
-Name=Please help us improve the software you use by filing a report at <a \
href="http://bugs.kde.org">http://bugs.kde.org</a>. Useful details include how to \
reproduce the error, documents that were loaded, etc. +Name=Please help us improve \
the software you use by filing a report at <a \
href="http://bugs.kde.org">http://bugs.kde.org</a>. Make sure that you describe in \
detail how to reproduce the crash and include all necessary information (e.g., the \
URL you visited in case of a browser crash or a document that caused an application \
to crash when you tried to open it).  Name[af]=Jy kan 'n foutrapport instuur vir die \
program. Kyk eers of dit by <a href=http://bugs.kde.org">http://bugs.kde.org</a> \
gelys is, anders kan jy die outeur direk kontak. Sluit asb. soveel inligting as \
moontlik in. As jy weet hoe om die probleem te reproduseer, sluit asb. ook 'n \
beskrywing daarvan in.  Name[bg]=Моля, помогнете ни да \
подобрим софтуера като съобщите за тази \
грешка на <a href="http://bugs.kde.org">http://bugs.kde.org</a>. \
Опишете как е възникнала грешката, кои \
документи са били заредени и т.н.  Name[ca]=Si us plau, \
ajudeu-nos a millorar el programari que utilitzeu omplint un informe a <a \
href="http://bugs.kde.org">http://bugs.kde.org</a>. Els detalls útils són com \
                reproduir l'error, el documents carregats,  etc.
Index: runtime/drkonqi/toplevel.cpp
===================================================================
--- runtime/drkonqi/toplevel.cpp	(revision 840918)
+++ runtime/drkonqi/toplevel.cpp	(working copy)
@@ -90,9 +90,10 @@
     mainLayout->addWidget(detailsCheckBox);
 
     m_detailDescriptionLabel = new QLabel(this);
-    m_detailDescriptionLabel->setText(i18n("Please attach the following information \
to your bug report:")); +    m_detailDescriptionLabel->setText(i18n("The following \
information should be added to your bug report. In many cases, you can make your bug \
report more useful by installing debugging packages, see <a \
href=\"http://techbase.kde.org/Development/Tutorials/Debugging/How_to_create_useful_crash_reports\">How \
to create useful crash reports</a> at KDE TechBase."));  \
                m_detailDescriptionLabel->setWordWrap(true);
     m_detailDescriptionLabel->setTextInteractionFlags(Qt::TextBrowserInteraction);
+    m_detailDescriptionLabel->setOpenExternalLinks(true);
     m_detailDescriptionLabel->setAlignment(Qt::AlignJustify);
     m_detailDescriptionLabel->setVisible(false);
     mainLayout->addWidget(m_detailDescriptionLabel);



>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread]