GNU bug report logs - #77109
[feature] [shepherd] Accept multiple services for actions

Previous Next

Full log

Message #29 received at 77109 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Tomas Volf <~@wolfsden.cz>
Cc: 77109 <at> debbugs.gnu.org, Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Subject: Re: bug#77109: [feature] [shepherd] Accept multiple services for
 actions
Date: Sat, 22 Mar 2025 19:49:34 +0100

Ludovic Courtès <ludo <at> gnu.org> skribis:

> Tomas Volf <~@wolfsden.cz> skribis:
>
>> Actually, now that I think about it, why are formal arguments for the
>> restart lambda `(running . args)'?  For example status has just `(_)'.
>> Neither `running' nor `args' are used in the lambda itself.
>>
>> Changing the formal arguments to simply _ would get us the same behavior
>> you get when you try to invoke e.g. status action with multiple
>> arguments (an error).
>
> I think it’s a bug.  Back in the day (commit
> a84ecf34be2a35e8b068d4232d8932acc5986c33) would pass those extra
> arguments to ‘start’:
>
>       ((restart)
>        (lambda (running . args)
>          (if running
>              (stop obj)
>              (local-output "~a was not running." (canonical-name obj)))
>          (start obj args)))
>
> Which is consistent and should probably be restored.

Done in 3ab3930a30495391e5176355e524a8dc026bd060.

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.