On Thu, Jun 23, 2011 at 7:46 AM, Nico -telmich- Schottelius < [email protected]> wrote:
> Nigel, > > Nigel Kersten [Thu, Jun 23, 2011 at 07:37:30AM -0700]: > > [...] > > > > It's simple enough at first, [...] > > > > that's exactly the use case for __addifnosuchline in cdist. > The type let's you do what you want to do right now, fast and > simple. That's what a feature I've always missed in puppet. > So it is really really simple to construct this, and the wiki link on Simple Text Patterns posted earlier shows how easy it is. > [...] I do think the "append if no such line" semantics aren't really > maintainable over > > time. > > If at any time in the future, I begin to add more and more lines > to the same file, I definitely should refacter and create a type > that handles managing this important file. > > For me it's mostly a "take the right tool for the right job" selection > here. > I understand, and I'd be really happy to see someone take those simple text patterns and create a module on the Forge for this collection of functionality so people can get it easily. I am hesitant for us to put something like this in the core distribution though, simply because we know it's a maintenance problem over time. I'd be much more keen to see us have a good key/value type and provider that is easier to maintain and achieves the same result as "append if no such line". -- You received this message because you are subscribed to the Google Groups "Puppet Users" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-users?hl=en.
