Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.

[Danny Milosavljevic <dannym@scratchpost.org>]

Hi,

On Tue, 21 Feb 2017 00:53:55 +0100
Clément Lassieur <clement@lassieur.org> wrote:
> +This is a list of two-element tuples, 

list of pairs.

>where each tuple contains the

each pair

> +subsystem name and a command (with optional arguments) to execute upon
> +subsystem request.
> +
> +The command @command{sftp-server} implements the SFTP file transfer
> +subsystem.
> +@example
> +'(("sftp" "/usr/libexec/sftp-server"))

Hmm, that is a list in there, not a two-element tuple:

scheme@(guile-user)> (car '(("A" "B")))
$1 = ("A" "B")
scheme@(guile-user)> (car (car '(("A" "B"))))
$3 = "A"
scheme@(guile-user)> (cdr (car '(("A" "B"))))
$2 = ("B")  <---- should be "B" without the parens for tuples

So I suggest either fix the example to be a pair ("sftp" . "/usr/libexec/sftp-server") or fix the docs.