Subject: Re: bug#48986: [PATCH] gnu: opendoas: Fix restricted path "safepath".

From: Brice Waegeneire <[email-masked]>
Date: Sat, 19 Jun 2021 23:48:39 +0200

Ludo’,

Thank you for the review.

Ludovic Courtès <[email-masked]> writes:

> Hi Brice,
>
> Brice Waegeneire <[email-masked]> skribis:
>
>> OpenDoas look for binaries in safepath when a rule specify a relative
>> command, such as “permit keepenv :wheel cmd guix”.
>>
>> * gnu/packages/admin.scm (opendoas)[phases]: Rename 'fix-install' phase
>> to 'pre-configure', run it before 'configure' and add a substitution for
>> safepath.
>
> You could maybe move the comment above to admin.scm, right above
> (substitute* "doas.c" …).

Done.

> Otherwise LGTM, thanks!

Pushed as eb939109b9c06a09e1534a403745bd362b653d95.

Cheers,
- Brice