On Mon, Aug 20, 2012 at 9:50 PM, Tim Mooney <tim.moo...@ndsu.edu> wrote: > In regard to: Re: [Puppet Users] Dynamic Lookup of facter variable., > Nigel...: > >>>> Facts were supposed to be able to be referenced as $factname without >>>> throwing the deprecation warning in your release, it's been fixed in >>>> later versions. >>> >>> >>> Are you saying that we do *not* need to reference facts as $::factname >>> in all our classes, not even in preparation for puppet 3.x? What if >>> we *are* referencing them that way, now? >> >> >> There's no harm in going that extra mile and being explicit that >> you're looking at a top scope variable fact, rather than a local >> variable of the same name, so you can continue to reference them as >> $::factname if you would like to do so. >> >> Requiring that wasn't an original goal, as it was deemed too high a >> cost for the most common case to force that rather ugly syntax on >> everyone. It was a bug in the deprecation warning code. >> >> Does that help? > > > It does help, thank you. > > It's one of those things that I wish I had known before I spent hours > changing our modules in preparation for what I thought was going to > be a requirement for puppet 3.x, but better late than never. :-) > > I appreciate the clarity you've provided on this. > > Tim
I do apologize for us messing up the deprecation warning. It's caused a lot of unnecessary churn for everyone, and it's frustrating for everyone involved :( -- You received this message because you are subscribed to the Google Groups "Puppet Users" group. To post to this group, send email to puppet-users@googlegroups.com. To unsubscribe from this group, send email to puppet-users+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-users?hl=en.