Issue #12371 has been updated by Daniel Pittman.
Nigel Kersten wrote: > Shouldn't we fix the autoloading rather than implementing a whitelist? Sorry, fix it how? Should we assume that something in `puppet/functions` was intended to be in `puppet/parser/functions`? Spell-check filenames and see if that makes it possible to autoload a file? Use the hamming distance between the name of a file and the name of a file we could autoload to figure out the user meant it to work, but got it wrong? > I know in the past I've used `modulepath/module/lib/foo` to sync arbitrary > code that something related to my Puppet infrastructure was using, even if I > had to manually load things from there. .....yes. > It feels like we should be able to just consistently load from the libdir, > and that's the more consistent approach. So, how is this different from the proposal that: * if the change is under a path we “own” like puppet or facter, we should warn. * we should support something like puppet/util that you can throw random junk into as a user. * we should not warn for foo or other top level namespaces we don’t own. ---------------------------------------- Bug #12371: pluginsync should warn when files are not in a special directory https://projects.puppetlabs.com/issues/12371#change-55245 Author: Patrick Carlisle Status: Accepted Priority: Normal Assignee: Patrick Carlisle Category: plug-ins Target version: Affected Puppet version: Keywords: Branch: If you put files in a module in a directory that doesn't get autoloaded there should be some sort of warning. -- 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.
