Issue #13312 has been updated by Hunter Haugen.
After poking at this with Nick he pointed out that it should also traverse inherited scopes before skipping to top/node scope. At the moment it just checks class-local scope and top/node scope, but not any inherited scopes. Will close the pull request for now until further changes can be pushed. ---------------------------------------- Bug #13312: Dynamic scoping deprecation warnings should be given at the correct times https://projects.puppetlabs.com/issues/13312#change-58480 Author: Hunter Haugen Status: In Topic Branch Pending Review Priority: Normal Assignee: Hunter Haugen Category: Target version: Affected Puppet version: 2.7.0 Keywords: scope Branch: https://github.com/hunner/puppet/tree/bug/2.7.x/incorrect_dynamic_scope-warnings Due to some misevaluated flow control, deprecation warnings for dynamic scoping are not being thrown correctly. The scoping module is due for an update and it would be helpful if these warnings were shown correctly. -- 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.
