Review Request: Shell integration: extend commands for syncing cwd

David Nolden david.nolden.kde at art-master.de
Sun Oct 7 17:28:16 UTC 2012



> On Oct. 7, 2012, 3:19 p.m., David Nolden wrote:
> > 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.
> 
> Ivan Shapovalov wrote:
>     The quotes around variable assignments and parameters fix potential bugs with paths containing spaces. (And the ones around network port/username variables are there just for the sake of completeness.)
>     IMO, quoting what one wants to be a single parameter/word is always a good idea when writing in shell... If you ACK, I'll post a separate patch (maybe for all shell scripts that we have).

Shell scripting isn't my strong side, I don't even like it, so patches improving the robustness are of course welcome. However it has to be made sure that no regressions are introduced, unfortunately there are no tests for the shell integration yet.


- David


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


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/0b337b06/attachment.html>


More information about the KDevelop-devel mailing list