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

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

Simon Streit <[email-masked]> skribis:

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

Just write “New file.”, that’s enough.

Please also add it to gnu/local.mk and to po/guix/POTFILES.in.

[...]

> +  #:export (samba-service-type
> +            samba-configuration
> +            samba-smb-conf
> +
> +            wsdd-service-type
> +            wsdd-configuration))

These two lines shouldn’t be here for now.  :-)

> +(define samba-service-type
> +  (service-type
> +   (name 'samba)
> +   (description "Samba")
> +   (extensions
> +    (list (service-extension shepherd-root-service-type
> +                             samba-shepherd-services)
> +          (service-extension activation-service-type
> +                             samba-activation)
> +          (service-extension profile-service-type
> +                             (compose list samba-configuration-package))))
> +   (default-value (samba-configuration))))

Please add a ‘description’ field with Texinfo markup (it’s the
description you see when running ‘guix system search’.)