Issue #12790 has been updated by Daniel Pittman. Status changed from In Topic Branch Pending Review to Merged - Pending Release Target version set to 2.0.0
Thanks so much for that submission. Sorry it took so long to merge. ---------------------------------------- Feature #12790: Facter should fail if a recursion is detected https://projects.puppetlabs.com/issues/12790#change-62642 Author: Stefan Schulte Status: Merged - Pending Release Priority: Normal Assignee: Category: Target version: 2.0.0 Keywords: Branch: https://github.com/puppetlabs/facter/pull/179 Affected Facter version: Facter already detects recursions (`fact a queries fact b queries fact a`) but instead of raising an exception or at least print a warning it will just return nil as the fact value and print a debug message (`Facter::Util::Fact#searching`) As seen in #11511 recursions can be hard to detect and can lead to unexpected results. In my opinion a recursion is never `expected behaviour`. -- 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.
