Issue #4732 has been updated by Markus Roberts.
Setting the target on a "Needs Design Decision" ticket sort of begs the question. The possible answers I have heard are: 1. This is correct, though possibly confusing behavior and should be documented 2. We should assume everything is puppet code unless it matches *.rb 3. As 2, but we should warn on "questionable" extensions such as .json, .htm, etc. (we'll need a list) 4. We should determine the type by inspection, not by the extension 5. We should assume type as in 2 but fall back to the other on failure Of these, only the first two are reasonably implementable for 2.6.1, and I slightly favour 2. ---------------------------------------- Bug #4732: Use of .pp/.rb extention to sniff file type breaks "everything is pp" assumption http://projects.puppetlabs.com/issues/4732 Author: Markus Roberts Status: Needs design decision Priority: Normal Assignee: Category: Target version: 2.6.1 Affected version: 2.6.1rc4 Keywords: Branch: R.I. writes: This fix breaks/improves import so that it will now only import files named .pp or .rb where in the past it would happily have imported anything matching the supplied glob: <pre> import "/etc/puppet/manifests/nodes/*" </pre> This now only imports .pp and .rb not * as it’s asked to do. -- 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.
