diff options
| author | Max Martin <max@puppetlabs.com> | 2011-04-26 11:44:16 -0700 |
|---|---|---|
| committer | Max Martin <max@puppetlabs.com> | 2011-04-26 11:44:16 -0700 |
| commit | 6064e8e37b0d6f9996921bfa598d5401a1f98537 (patch) | |
| tree | 0db9dc7cea499ba76d609be93e90219924961157 /lib/puppet/parser | |
| parent | 67a86554fff18cd75744ea987ea2361f495efc4c (diff) | |
| download | puppet-6064e8e37b0d6f9996921bfa598d5401a1f98537.tar.gz puppet-6064e8e37b0d6f9996921bfa598d5401a1f98537.tar.xz puppet-6064e8e37b0d6f9996921bfa598d5401a1f98537.zip | |
(#7101) Fix template error messages in Ruby 1.8.5
lib/puppet/parser/templatewrapper.rb#script_line was calling .first on a
String object, which in Ruby > 1.8.5 will return the entire string, but
in 1.8.5 will cause an exception. This change (proposed by Markus
Roberts) removes the call to .first and adds a condition for when the
template error involves the file not being found.
Reviewed-by: Jesse Wolfe
Diffstat (limited to 'lib/puppet/parser')
| -rw-r--r-- | lib/puppet/parser/templatewrapper.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/puppet/parser/templatewrapper.rb b/lib/puppet/parser/templatewrapper.rb index 180a03dc9..27d75bf92 100644 --- a/lib/puppet/parser/templatewrapper.rb +++ b/lib/puppet/parser/templatewrapper.rb @@ -20,7 +20,7 @@ class Puppet::Parser::TemplateWrapper def script_line # find which line in the template (if any) we were called from - caller.find { |l| l =~ /#{file}:/ }.first[/:(\d+):/,1] + (caller.find { |l| l =~ /#{file}:/ }||"")[/:(\d+):/,1] end # Should return true if a variable is defined, false if it is not |