Re: [PATCH] gnu: Add Kerberos client service.

[Ludovic Courtès]

Hi!

John Darrington <address@hidden> skribis:

> * doc/guix.texi: New subsubheading: Krb5 Service
> * gnu/services/kerberos.scm: New variable: krb5-service-type.

Please see ‘git log’ on how to introduce new sections and variables.

> address@hidden Krb5 Service
> +
> +The krb5 service provides the configuration for Kerberos clients, using
> +the MIT implementation of the Kerberos protocol address@hidden

Maybe add: “ (the @code{mit-krb5} package).”

Also it would be good to add one or two sentences of context, like:

  Programs using the client library of @code{mit-krb5} expect such
  configuration files in @file{/etc/something} and read them to set up
  their Kerberos frobbing thingie.  System administrators typically
  define the appropriate configuration based on the network's Kerberos
  foobar.

Well, something like that.  :-)

> address@hidden {Scheme Variable} krb5-service-type
> +A service type for Kerberos 5 clients.

s/Kerberos 5/@code{mit-krb5}/

> address@hidden
> +(service krb5-service-type (krb5-configuration
> +                             (default-realm "EXAMPLE.COM")

Please move the ‘krb5-configuration’ to the next line where it will be
aligned below ‘krb5-service-type’.  This will ensure correct PDF/PS/DVI
output as well as below-80 Info lines.

> address@hidden @code{allow-weak-crypto?} (default: @code{#f})
> +If this flag is #t then services which only offer encryption algorithms

@code{#t}

> +(define-syntax define-configuration
> +  (lambda (stx)
> +    (define (id ctx part . parts)

Nack!  We already have two copies of that one, we won’t add a third
one.  :-)

Fortunately, Julien submitted a patch to factorize it, so hopefully you
can use the factorize ‘define-configuration’ in a day or two.

> +;; For a more detailed explanation of these fields see man 5 krb5.conf
> +(define-configuration krb5-realm
> +  (name
> +   (string #f)
> +   "The name of the realm.")

Really cool.

> +;;  LocalWords:  Guix

Maybe we can avoid that, but no big deal.

So OK with changes along these lines when Julien’s factorization has
landed.

Thank you!

Ludo’.