On Oct 26, 2009, at 8:43 AM, Brice Figureau wrote:
>
> On Mon, 2009-10-26 at 08:40 -0700, Luke Kanies wrote:
>> I'd like to see a test in this, and probably a comment as to why that
>> line is there.
>
> I'm not even sure this is the right way to fix the issue...
> I know I suggested Volcane this way of hotfixing the issue, but I
> don't
> really understand why the functions are not known to each other in
> puppet while they are in the master.
As discussed on IRC, it's not a question of executables, it's a
question of autoloading - you're unlikely to have much of a manifest
in your server without using (and thus autoloading) 'include' at least
once, but 'puppet' is usually used for sample manifests that might not
use 'include' at all.
So, this patch just forces the load. You could probably call
something like this instead:
Puppet::Parser::Functions.loader.load(:include), but the loader might
be private, etc. They're functionally equivalent.
--
America believes in education: the average professor earns more money
in a year than a professional athlete earns in a whole week.
-- Evan Esar
---------------------------------------------------------------------
Luke Kanies | http://reductivelabs.com | http://madstop.com
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
