On Thu, Feb 26, 2015 at 09:07:50AM +0100, yy wrote:
> On 26 February 2015 at 07:18, David Phillips <dbphillip...@gmail.com> wrote:
> > Being the author of the initial patch, Marc has reminded me to submit
> > any feedback I have on the abduco_force patch, which will supersede my
> > contribution. I did initially think of using a -f option, but didn't
> > think there would be a need for any other option to be forced. This is
> > how I ended up using a capital C rather than a -f option.
> 
> Thank you for the feedback.
> 
> The reason I find the -f option useful with -A is that now I can run
> abduco -f -A session, for example when I start a ssh, to always get a
> running session (either attaching, creating a new one or finishing a
> terminated one and creating another). A small convenience.

I see how this can be useful in some cases. However it has become quite
a mess. Consider the following:

 abduco -n demo
 abduco -f -A demo

with your patch this first attached to the session and upon completion
created another new session. I don't think this was intended? I pushed
a fix for this roughly based on your patch.

Due to some unrelated changes you should remove or chmod 700 your 
$HOME/.abduco folder. 

Please provide feedback whether this works as expected.

Thanks

-- 
 Marc André Tanner >< http://www.brain-dump.org/ >< GPG key: CF7D56C0

Reply via email to