Bug/2.7.x/7991 and 4549 autoloading for custom functions#124
Merged
slippycheeze merged 1 commit intopuppetlabs:2.7.xfrom Sep 16, 2011
slippycheeze:bug/2.7.x/7991-and-4549-autoloading-for-custom-functions
Merged
Bug/2.7.x/7991 and 4549 autoloading for custom functions#124slippycheeze merged 1 commit intopuppetlabs:2.7.xfrom slippycheeze:bug/2.7.x/7991-and-4549-autoloading-for-custom-functions
slippycheeze merged 1 commit intopuppetlabs:2.7.xfrom
slippycheeze:bug/2.7.x/7991-and-4549-autoloading-for-custom-functions
Conversation
Only a small subset of Puppet functions were available on the scope in
templates. This had people doing workarounds like:
inline_template("<%= Puppet::Parser::Functions.autoloader.loadall; scope.function_extlookup(['hello world']) %>")
Rather than need to know these workarounds and need to load all
templates, the scope object now uses method_missing to find the function
if avaialable.
Paired-with: Nick Lewis <nick@puppetlabs.com>
slippycheeze
added a commit
that referenced
this pull request
Sep 16, 2011
…utoloading-for-custom-functions Bug/2.7.x/7991 and 4549 autoloading for custom functions Merged for the second time, because something in the murky past ate the original merge into the main tree, done by mmrobins.
melissa
pushed a commit
to melissa/puppet
that referenced
this pull request
Mar 30, 2018
(PCP-28) Add interpreter field to module config options
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit fixed an earlier bug, but was somewhere lost from the main tree after Matt announced it merged. I am pushing it back in, as it fixes a real problem, and is current.
Only a small subset of Puppet functions were available on the scope in
templates. This had people doing workarounds like:
Rather than need to know these workarounds and need to load all
templates, the scope object now uses method_missing to find the function
if avaialable.
Paired-with: Nick Lewis nick@puppetlabs.com