Review Request: Shell integration: extend commands for syncing cwd

Andreas Pakulat apaku at gmx.de
Mon Sep 24 08:32:34 UTC 2012



> On Sept. 24, 2012, 12:30 a.m., Aleix Pol Gonzalez wrote:
> > shell/projectcontroller.h, line 86
> > <http://git.reviewboard.kde.org/r/106508/diff/1/?file=86379#file86379line86>
> >
> >     Why don't you use the findProjectForUrl thing?

Because thats not doing the same thing. Look at the implementation, the prefix is just a text, not an url.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106508/#review19370
-----------------------------------------------------------


On Sept. 19, 2012, 6:10 p.m., Ivan Shapovalov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106508/
> -----------------------------------------------------------
> 
> (Updated Sept. 19, 2012, 6:10 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> This adds 2 new commands for syncing the shell working directory (in kdevplatform_shell_environment.sh).
> These are
> - "project!", which navigates to the root directory of a project, and
> - "bdir!", which navigates to the build directory of a project (if one is returned by the build manager).
> 
> These commands take an optional argument - project name, which (if exists) is handled as a target project name prefix and is collated in case-insensitive mode.
> Thus, to navigate to kdevplatform's root directory one may type "project! kdevpl".
> If that argument is not given, the project of currently open document is selected.
> 
> Reason: I just found these commands useful. Maybe someone else also will.
> 
> 
> Diffs
> -----
> 
>   shell/projectcontroller.h e21345f 
>   shell/projectcontroller.cpp e5fe56f 
>   util/kdevplatform_shell_environment.sh d428f6c 
> 
> Diff: http://git.reviewboard.kde.org/r/106508/diff/
> 
> 
> Testing
> -------
> 
> Used the newly-added commands for a sufficiently long time period (~2 weeks).
> 
> 
> Thanks,
> 
> Ivan Shapovalov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120924/b60a1c19/attachment.html>


More information about the KDevelop-devel mailing list