Review Request 123675: Add an "Allow Multiple Instances" checkbox in the executable launch configuration

Aleix Pol Gonzalez aleixpol at kde.org
Thu May 14 00:12:38 UTC 2015



> On May 7, 2015, 6:26 p.m., Andreas Pakulat wrote:
> > I'd consider this a saner default and personally would likely remove the preference option and just change the behavior except for one thing: If you notice an instance is already started ask the user wether he wants to kill it or not. I run into a similar thing every once in a while with Eclipse starting an Eclipse application. Since that app uses a fixed workspace and a workspace can only ever be used by one instance of Eclipse I get a notification when I try to start the second instance. Most of the time I'm glad Eclipse doesn't just kill the already running instance to start a new one, since I simply forgot I still had it running.

Works for me, I'll drop the GUI option.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123675/#review80048
-----------------------------------------------------------


On May 7, 2015, 5:02 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123675/
> -----------------------------------------------------------
> 
> (Updated May 7, 2015, 5:02 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> One of the things that bothered me the most when running applications on KDevelop is that eventually I'd had different instances of the program I'm working on at the same time, forcing me to kill them when I realized that.
> 
> What this patch those is add a checkbox setting that lets the user specify whether she wants to have the separate process running.
> 
> 
> Diffs
> -----
> 
>   plugins/execute/nativeappconfig.cpp d5f4e83 
>   plugins/execute/iexecuteplugin.h c6b7fd7 
>   plugins/execute/executeplugin.cpp 770e33e 
>   plugins/execute/executeplugin.h 3e379b1 
>   plugins/execute/nativeappconfig.ui bb8b461 
>   plugins/execute/nativeappjob.h 603f7b5 
>   plugins/execute/nativeappjob.cpp f8e2ba0 
>   project/builderjob.cpp 826f348 
> 
> Diff: https://git.reviewboard.kde.org/r/123675/diff/
> 
> 
> Testing
> -------
> 
> Played with it using kdevelop.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

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


More information about the KDevelop-devel mailing list