[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 4/4] services: openssh: Add 'subsystems' option. |
Date: |
Thu, 16 Mar 2017 11:03:27 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hi Clément,
Clément Lassieur <address@hidden> skribis:
> I think there is a misunderstanding. I didn't want to push this because
> it was not tested, and because subsystems are often useless without
> Match, and Match is unsupported. The pair/list thing is not the
> problem.
>
> And I was waiting for ng0 to test, because he asked for this patch (see
> http://lists.gnu.org/archive/html/guix-devel/2017-02/msg00906.html), so
> I thought he was able to test.
>
> If ng0 still needs the patch and confirms that it works well, I'm
> willing to update it. Otherwise, let's drop it.
The patch looks like a useful addition. It would be sad to drop it, no?
If you could add a test to (gnu tests ssh) that would be perfect, but
otherwise it LGTM. Perhaps the default value for ‘subsystems’ should
include internal-sftp though, as users probably expect sftp to just work.
I don’t have any issue with using two-element lists; I think the syntax
for pairs can be a bit confusing for newcomers to it’s probably better
to avoid it in service configuration.
Thanks,
Ludo’.
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., Danny Milosavljevic, 2017/03/02
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., Clément Lassieur, 2017/03/02
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., ng0, 2017/03/05
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., Danny Milosavljevic, 2017/03/07
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., Clément Lassieur, 2017/03/07
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.,
Ludovic Courtès <=
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., ng0, 2017/03/16
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., Clément Lassieur, 2017/03/16
- Re: [PATCH 4/4] services: openssh: Add 'subsystems' option., John Darrington, 2017/03/17
- grammar usage (was: Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.), ng0, 2017/03/17
- Re: grammar usage (was: Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.), John Darrington, 2017/03/17
- Re: grammar usage (was: Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.), ng0, 2017/03/17
- Re: grammar usage (was: Re: [PATCH 4/4] services: openssh: Add 'subsystems' option.), John Darrington, 2017/03/17
- Re: grammar usage, Andy Wingo, 2017/03/17
- Re: grammar usage, John Darrington, 2017/03/17
- Re: grammar usage, Andy Wingo, 2017/03/17