[augeas-devel] [PATCH 1 of 2] Add puppet.aug and associated test

[Raphaël Pinson]

On Wed, Aug 13, 2008 at 8:41 PM, David Lutterkort <dlutter at redhat.com>wrote:

> On Wed, 2008-08-13 at 18:54 +0200, Raphael Pinson wrote:
> > # HG changeset patch
> > # User Raphael Pinson <raphink at gmail.com>
> > # Date 1218646423 -7200
> > # Node ID 6f492eaaa453856246c5ffa79ffd1c8e06939261
> > # Parent  98e1bdd4af843caab45a32d0120c6c5497260f2d
> > Add puppet.aug and associated test
>
> Nice :) ACK, committed.
>
> One only slightly related thing about Inifile:
>
> > --- /dev/null Thu Jan 01 00:00:00 1970 +0000
> > +++ b/lenses/tests/test_puppet.aug    Wed Aug 13 18:53:43 2008 +0200
> > @@ -0,0 +1,31 @@
> > +module Test_puppet =
> > +
> > +   let conf = "
> > +[main]
> > +logdir=/var/log/puppet
> > +
> > +[puppetd]
> > +server=misspiggy.network.com
> > +"
> > +
> > +   test Puppet.lns get conf =
> > +      {}
> > +      { "section" = "main"
> > +         { "logdir" = "/var/log/puppet" }
> > +         {} }
> > +      { "section" = "puppetd"
> > +         { "server" = "misspiggy.network.com" } }
>
> Wouldn't it be nicer to use the titles of sections as their key instead
> of calling everything section ? That would make navigating the tree
> easier ... for cases where section names are too horrendous (Php ?) the
> way things are right now are preferrable.
>


That's funny! I thought about that just today when I was making this lens,
and then I thought that I would look at it later because of the php.aug
mess, and how some other INI Files like dput have unpredictable section
titles, too, except for [DEFAULT].

I'll have a look at that, it definitely shouldn't be too hard.



Raphael
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/augeas-devel/attachments/20080813/9f4be48a/attachment.htm>