diff options
author | Luke Kanies <luke@puppetlabs.com> | 2010-04-13 08:51:17 -0700 |
---|---|---|
committer | Luke Kanies <luke@puppetlabs.com> | 2010-04-13 08:51:17 -0700 |
commit | da00f682ce0d1778183f9cf7dbe6d2e133b11841 (patch) | |
tree | 08822d84323dcb1a659ef7badd78f7526bb1681a /lib | |
parent | 9792915481ed757c56768f8f5e4fd100217b7232 (diff) | |
download | puppet-da00f682ce0d1778183f9cf7dbe6d2e133b11841.tar.gz puppet-da00f682ce0d1778183f9cf7dbe6d2e133b11841.tar.xz puppet-da00f682ce0d1778183f9cf7dbe6d2e133b11841.zip |
Making a Puppet::Module test more resilient
It would fail if a directory unexpectedly existed.
Signed-off-by: Luke Kanies <luke@puppetlabs.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/puppet/module.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/puppet/module.rb b/lib/puppet/module.rb index f30b862ef..e598a8e88 100644 --- a/lib/puppet/module.rb +++ b/lib/puppet/module.rb @@ -126,10 +126,10 @@ class Puppet::Module return find_init_manifest unless rest # Use init.pp rest ||= "init.pp" - p = File::join(path, MANIFESTS, rest) - result = Dir.glob(p).reject { |f| FileTest.directory?(f) } + full_path = File::join(path, MANIFESTS, rest) + result = Dir.glob(full_path).reject { |f| FileTest.directory?(f) } if result.size == 0 and rest !~ /\.pp$/ - result = Dir.glob(p + ".pp") + result = Dir.glob(full_path + ".pp") end result.flatten.compact end |