Subject: Re: [bug#66360] [PATCH] doc: Use herd for cerbot deploy hook example.
From: Ludovic Courtès <[email-masked]>
To: Bruno Victal <[email-masked]>
Cc: [email-masked]
Date: Sat, 14 Oct 2023 22:28:59 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi,

Bruno Victal <[email-masked]> skribis:

> * doc/guix.texi (Certificate Services): Replace PID file based example with
> one using (gnu services herd). Rename %nginx-deploy-hook to
> %certbot-deploy-hook.

Applied, thanks!

> +(define %certbot-deploy-hook
> +  (program-file "certbot-deploy-hook.scm"
> +    (with-imported-modules '((gnu services herd))
> +      #~(begin
> +          (use-modules (gnu services herd))
> +          (with-shepherd-action 'nginx ('reload) result result)))))

Should we make it the default ‘deploy-hook’ value in
<certificate-configuration>?  I’ve always been frustrated that
‘certbot-service-type’ doesn’t quite work out of the box.

Ludo’.