Review Request: Shell integration: extend commands for syncing cwd

[David Nolden]

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


I think this is useful, but I'd prefer the "bdir!" command to do a bidirectional switch between build- and project-dir, keeping the relative sub-directory alive, eg.:
kdevplatform/shell$ b! -> kdevplatform_builddir/shell
and
kdevplatform_builddir/shell$ b! -> kdevplatform/shell

It could keep the semantics if the current path is not within the projects dir, eg.:
kdevplatform/shell$ b! kdevelop -> kdevelop_builddir

And the current behavior could be achieved through p!; b!

Also I think that the explanation about prefixes in "help!" is cluttering the output a bit too much, maybe this should be moved into a "help! project" subsection.

- David Nolden


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/4fb678d9/attachment.html>