Hi Danny,
Danny Milosavljevic <dan...@scratchpost.org> writes:
> Hi,
>
> On Tue, 21 Feb 2017 00:53:55 +0100
> Clément Lassieur <clem...@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.
Ok. I'll correct this later, for now I don't want to push this patch,
I'm waiting for a confirmation from ng0.
Thanks for the review!
