Issue #4206 has been updated by Brice Figureau.

Status changed from Unreviewed to Ready for Testing
Assigned to set to Brice Figureau


----------------------------------------
Bug #4206: import "path/*" tries to import files twice
http://projects.puppetlabs.com/issues/4206

Author: Brice Figureau
Status: Ready for Testing
Priority: Normal
Assigned to: Brice Figureau
Category: compiler
Target version: 2.6.0
Affected version: 2.6.0rc1
Keywords: 
Branch: http://github.com/masterzen/puppet/tree/tickets/master/4206


Using the following manifests snippet:

<pre>
import "os/*"
</pre>

With the following filenames in os/:
* debian.pp
* redhat.pp


Uses the following ruby code (in lib/puppet/parser/files.rb):
<pre>
Dir.glob("/path/to/os/*{,.pp,.rb}")
</pre>

which consequently produces:
<pre>
["/path/to/os/debian.pp", "/path/to/os/redhat.pp", "/path/to/os/debian.pp", 
"/path/to/os/redhat.pp"]
</pre>

Unfortunately since it is forbidden to import twice the same manifests (see 
#4205), puppet aborts with an "import loop error".

Rewriting the glob pattern to '{.pp,.rb}' (notice the missing leading comma), 
fix the issue.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" 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-bugs?hl=en.

Reply via email to