[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Danny Milosavljevic
Subject: Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.
Date: Tue, 7 Mar 2017 21:49:59 +0100

Hi ng0,

On Sun, 5 Mar 2017 14:50:26 +0000
ng0 <address@hidden> wrote:

> What I take from the discussion is, all is good to go except for
> subsystems. I'm okay with reviewing subsystems as an individual patch
> later on. For me this works. Push the 3 patches, and send the subsystems
> one later as a new discussion-bug.

The 3 were pushed to master.

Patch 4 not yet. So let's discuss.

I have no preference for pairs or lists - it's just that the documentation 
should say what it actually expects - because the user has to write the form 

  Pair: '(a . b)

  List: '(a b)

Those are not compatible with each other.

(I think as the patch is written now it expects lists)

And I'm against calling pairs "two-element tuple"s. It reminds me of these math 
joke equations which write the value 2 in a really complicated way (but 
correctly) :)

And lists are definitely not two-element tuples. That would be seriously 

What do you think?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]