[Pkg-puppet-devel] [facter] 128/352: use instance_variable_get instead of eval

Stig Sandbeck Mathisen ssm at debian.org
Sun Apr 6 22:21:38 UTC 2014


This is an automated email from the git hooks/post-receive script.

ssm pushed a commit to branch master
in repository facter.

commit 190825c1fc7cf3836a60d72ccf6deff6e03ea440
Author: Clay Caviness <clay at boobah.com>
Date:   Tue Aug 20 09:19:42 2013 +0100

    use instance_variable_get instead of eval
---
 ext/osx/preflight.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ext/osx/preflight.erb b/ext/osx/preflight.erb
old mode 100755
new mode 100644
index ab7d9af..994cbdd
--- a/ext/osx/preflight.erb
+++ b/ext/osx/preflight.erb
@@ -9,7 +9,7 @@
 # when being installed to volumes other than the current OS.
 
 <%- ["@apple_libdir", "@apple_sbindir", "@apple_bindir", "@apple_docdir", "@package_name"].each do |i| -%>
-  <%- val = eval(i) -%>
+  <%- val = instance_variable_get(i) -%>
   <%- raise "Critical variable #{i} is unset!" if val.nil? or val.empty? -%>
 <%- end -%>
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-puppet/facter.git



More information about the Pkg-puppet-devel mailing list