Subject: Re: bug#54561: [PATCH 0/4] Add service declarations for Samba

From: Ludovic Courtès <[email-masked]>
Date: Fri, 08 Apr 2022 23:43:33 +0200

Simon Streit <[email-masked]> skribis:

> * gnu/services/samba.scm (<wsdd-configuration>): New record.
> (wsdd-service-type): New variable.
> (wsdd-shepherd-services): New procedure.

Just “New file.” and add it to gnu/local.mk.

> +(define wsdd-shepherd-service
> +  (match-lambda
> +    (($ <wsdd-configuration> package ipv4only? ipv6only? chroot hoplimit
> +                             interfaces uuid-device domain hostname
> +                             preserve-case? workgroup)

Please use ‘match-record’ instead; it is less error-prone.

> +(define wsdd-service-type
> +  (service-type
> +   (name 'wsdd)
> +   (description "Web Service Discovery Daemon")

Please write full sentences in ‘description’, possibly with Texinfo
markup.

Ludo’.