Angelos -- I poked at it a bit but didn't have any blinding insights. A few >> observations: >> >
> Overall it looks pretty good, but I suspect I'm missing something. >> >> What are you using for a test case? Can you update the ticket with ajust >> me. > > bit more information about how you're trying it, and maybe that will >> suggest something. >> > > Hrm. Now I came in for work and tried it all again, it works (i.e. it > produces a warning) as expected. I guess it was a PEBKAC error, probably > missed an install or server restart and have been chasing ghosts since. So > I'll attach my patch to the bug report and will mail puppetlabs the signed > CLA later today. > No worries; I'm glad it wasn't just me failing to reproduce. :) Thanks for your help, You're quite welcome, and thank you for the patch. These sort of "simplify and increase consistency" changes are really good for the code base and make Puppet more pleasant to use and to work on. -- Markus ----------------------------------------------------------- The power of accurate observation is commonly called cynicism by those who have not got it. ~George Bernard Shaw ------------------------------------------------------------ -- You received this message because you are subscribed to the Google Groups "Puppet Developers" 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-dev?hl=en.
