Review Request 118954: Set directory to current working directory when executing "Open With..." dialog

Dawit Alemayehu adawit at kde.org
Sat Jun 28 12:32:41 BST 2014



> On June 28, 2014, 8:31 a.m., David Faure wrote:
> > kio/kio/krun.cpp, line 733
> > <https://git.reviewboard.kde.org/r/118954/diff/2/?file=285056#file285056line733>
> >
> >     now this line is going to be called if urls is empty, even with an empty service.path() .....
> >     
> >     This should be:
> >     
> >     if !service.path.isEmpty
> >       use that
> >     else if !urls.isEmpty
> >       use first url
> >

D'oh. That was dumb. Fixed.


- Dawit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118954/#review61122
-----------------------------------------------------------


On June 28, 2014, 2:09 a.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118954/
> -----------------------------------------------------------
> 
> (Updated June 28, 2014, 2:09 a.m.)
> 
> 
> Review request for kdelibs and David Faure.
> 
> 
> Bugs: 142597
>     http://bugs.kde.org/show_bug.cgi?id=142597
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> The attached patch sets the current directory to the current working directory (read: directory in which the open with dialog was run) whenever the KService path is empty. That way we tell KProcess in the context of the current directory instead of the default one it uses, $HOME.
> 
> 
> Diffs
> -----
> 
>   kinit/klauncher.cpp 6c71e99 
>   kio/kio/krun.cpp 590fcf8 
> 
> Diff: https://git.reviewboard.kde.org/r/118954/diff/
> 
> 
> Testing
> -------
> 
> Tested with the example provided in the bug report. Verified the output of the compile process is in the current working directory instead of $HOME.
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140628/d8c2fe50/attachment.htm>


More information about the kde-core-devel mailing list