Review Request 109213: Allow opening of session by project name on command line

Andreas Pakulat apaku at gmx.de
Wed Mar 6 19:19:29 UTC 2013



> On March 6, 2013, 7:13 p.m., Andreas Pakulat wrote:
> > app/main.cpp, line 332
> > <http://git.reviewboard.kde.org/r/109213/diff/1/?file=116257#file116257line332>
> >
> >     This is actually not matching the project name. It matches the projects .kdev4-filename. Many of my projects have a name different from the project filename.
> >     
> >     I think you should rather match against what the session-listing produces as list-of-projects since that has the names.

Nevermind, project names are not available at that point yet.


- Andreas


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


On Feb. 28, 2013, 8:38 p.m., Eli MacKenzie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109213/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2013, 8:38 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> `kdevelop -s NAME` only allows the opening by hash or session name, even for unnamed sessions. Since it appears to be pretty easy to leave sessions unnamed, its handy to be able to specify by project. If any session shares a project name, the session will be selected by name. Only in the case where the name does not refer to a session will a project name be used.
> 
> 
> Diffs
> -----
> 
>   app/main.cpp 9642c2f2dec5547e8eeacb9cfa837667d9401e95 
> 
> Diff: http://git.reviewboard.kde.org/r/109213/diff/
> 
> 
> Testing
> -------
> 
> As is obvious: session selection by project name didn't work before the patch.
> 
> 
> Thanks,
> 
> Eli MacKenzie
> 
>

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


More information about the KDevelop-devel mailing list