Review Request: Shell integration: extend commands for syncing cwd

David Nolden david.nolden.kde at art-master.de
Sun Oct 7 15:19:38 UTC 2012


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


I've implemented this a bit differently now so that it has the bidirectional switching functionality. Thank you for the patch. You did many changes in parameter-escaping, can you explain these a bit? If those fix some bugs, please submit them separately based on the current trunk.

- David Nolden


On Sept. 27, 2012, 5 p.m., Ivan Shapovalov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106508/
> -----------------------------------------------------------
> 
> (Updated Sept. 27, 2012, 5 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), using shell's cwd (if in source tree) to find the corresponding subdirectory in the build tree.
> 
> 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 bc557bd 
>   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/20121007/4fcfaac1/attachment.html>


More information about the KDevelop-devel mailing list