On Thu, Feb 26, 2015 at 09:07:50AM +0100, yy wrote:
> On 26 February 2015 at 07:18, David Phillips <dbphillipsnz_AT_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
Received on Thu Feb 26 2015 - 15:59:21 CET