[augeas-devel] sshd lens bugfix

Matthew Palmer matt at anchor.net.au
Mon Feb 1 22:59:23 UTC 2010


On Mon, Feb 01, 2010 at 09:39:07AM -0800, David Lutterkort wrote:
> On Sat, 2010-01-30 at 22:57 +1100, Matthew Palmer wrote:
> > Apparently nobody noticed that SSH subsystem commands can quite validly have
> > spaces in them ("with optional arguments"); the patch below fixes that
> > problem.
> 
> ACK. While we're at it, should we split the arguments out from the
> command ? I.e., rather than parsing "Subsystem foo cmd arg1 arg2\n" into
> 
>   { "Subsystem" { "foo" = "cmd arg1 arg2" } }
> 
> parse it into
> 
>   { "Subsystem" { "foo" = "cmd" { "arg" = "arg1" } { "arg" = "arg2" } } }

Probably.  It doesn't bother me either way at the moment, though -- I only
stumbled across this bug because I was trying to change a "simple" value in
a config file the existing lens didn't parse properly (due to a long-ago
manual change to the Subsystem config item).

- Matt




More information about the augeas-devel mailing list