Re: [Chicken-hackers] [PATCH] Add support for sudo alternatives

[Evan Hanson]

Hi all,

On 2016-03-27 21:03, Peter Bex wrote:
> On Fri, Mar 25, 2016 at 04:11:41PM +0200, Timo Myyrä wrote:
> > Here is an simple patch to add support for alternative commands than sudo.
> > OpenBSD doesn't have sudo in base install so this allows chicken-install to 
> > work
> > with OpenBSD doas (and others like it).
> > The idea is that if user provides SUDO=/usr/bin/doas in env, then
> > chicken-install will run that command when using `-s' flag.
> 
> Thanks for the patch.  Attached is a signed-off version, with some
> minor cleanups.

That's a nice change indeed, pushed. Thanks, Timo and Peter.

Note that the current implementation doesn't allow for Matt's example
use case directly, since it shell-quotes the $SUDO command and thus
won't allow passing option flags:

On 2016-03-25  8:45, Matt Welland wrote:
> Thanks for this. Where faceless shared adminstration accounts are used this
> will be very helpful. From looking at the change setting SUDO to a command
> that includes spaces should work fine so something like:
>·
> SUDO='sudo -u faceless' chicken-install -s someegg
>·
> should work to build as current user and install as user faceless on Linux.
> Nice.

However, you can work around this with a zero-arguments script that does
the same.

Cheers,

Evan

signature.asc
Description: PGP signature