Bug 182384

Summary: KBugReport does not send mails
Product: [Unmaintained] kdelibs Reporter: Ezequiel R. Aguerre <ezeaguerre>
Component: kbugreportAssignee: David Faure <faure>
Status: RESOLVED FIXED    
Severity: normal CC: faure
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: This patch should fix it.

Description Ezequiel R. Aguerre 2009-01-30 06:58:13 UTC 
Version:            (using KDE 4.2.0)
OS:                Linux
Installed from:    Fedora RPMs

I'm writing an application and KBugReport is not sending the bugs to the mail address I've configured.
It seems the problem is you are passing the arguments with extra spaces, like this: " --subject ", it should be: "--subject".
And when you finish writing data to the process you should close the channel, otherwise it will keep waiting forever.
The exit status of ksendbugmail is QProcess::NormalExit, but the exit status is 254, so you won't catch that with proc.exitCode() == 1, it's probably safer to do proc.exitCode() != 0.
Comment 1 Ezequiel R. Aguerre 2009-01-30 07:01:14 UTC 
Created attachment 30738 [details]
This patch should fix it.

This patch should fix the problem. However, it does nothing about the comparison proc.exitCode() == 1, I think you should have a good reason to make it that way.
Comment 2 David Faure 2009-08-21 16:36:11 UTC 
Indeed things are broken in this area. I see a ksendbugmail crash, an smtp relay error not reported up to the dialog, and your patch looks correct too. Assigning to me, but I have to go so I'll look into all this next week or the week after.
Comment 3 David Faure 2009-08-27 16:24:17 UTC 
SVN commit 1016321 by dfaure:

Fix error handling inside ksendbugmail; e.g. when no internet available. Incomplete qt3 porting.
CCBUG: 182384


 M  +8 -7      main.cpp  
 M  +12 -14    smtp.cpp  
 M  +1 -0      smtp.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1016321
Comment 4 David Faure 2009-08-27 16:40:59 UTC 
SVN commit 1016338 by dfaure:

Commit patch by Ezequiel R. Aguerre to make kbugreport work again when sending bugs by email.
It was another qt3 porting bug: it was passing " --subject " instead of "--subject".
And closeWriteChannel is necessary otherwise it will keep waiting forever.
CCBUG: 182384


 M  +8 -4      kbugreport.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1016338
Comment 5 David Faure 2009-08-27 16:45:37 UTC 
SVN commit 1016340 by dfaure:

The exitcode was 254 when passing wrong args to ksendbugmail indeed, so let's not just check for 1, but for != 0.
BUG: 182384


 M  +1 -1      kbugreport.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1016340