On Sunday 28 December 2014 18:56:26 Gregor Mi wrote: > > Hmm, yeah, but if KRun displays error messages on its own, the app has > > nothing else to do. > > (No error messages were displayed to the user. At least I saw none. ...But > the error handling is a separate topic I think; or is solved by your fix.)
I added error handling now, kioexec brings up a messagebox if given an invalid URL or non-existing URL as argument. > > It's a command-line parsing problem. > > I'll add unittests for KRun::run(service, urls) with and without > > tempFiles, > > and then I'll fix it :-) > > Great. :-) Fixed, my unittest works. Turns out it wasn't a command-line parsing problem, at least not the way I thought. These arguments were meant for gwenview, not for kioexec. Can you try again after updating kio? -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE Frameworks 5 >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
