Issue #13952 has been updated by Daniel Pittman. Category set to inventory service Status changed from Unreviewed to Needs Decision Assignee set to Pieter van de Bruggen
This is still true in Telly as it stands today. Pieter, I think you are the best placed person to drive the decision about what should happen here, and how it works. (Ultimately, this is because there are only strings in Puppet, not other data types, no matter how much it kind of looks like there are - or, at least, how we stringify types in a liberal fashion.) ---------------------------------------- Bug #13952: 'false' matches . but not false or default. https://projects.puppetlabs.com/issues/13952#change-61077 Author: Jos Boumans Status: Needs Decision Priority: Normal Assignee: Pieter van de Bruggen Category: inventory service Target version: Affected Puppet version: 2.6.16 Keywords: Branch: See the below script. In my opinion the right order of evaluation here would be: 1) A direct lookup - in this case, the key 'false' 2) Any regular expressions - order is up in the air; declaration order perhaps? 3) default - if all else fails, that should be the right answer. Note, that in my opinion, false should definitely NOT match /./. # cat false.pp $x = false $y = $x ? { /./ => 1, false => 2, default => 42, } notice( $x ) notice( $y ) # puppet apply false.pp notice: Scope(Class[main]): false notice: Scope(Class[main]): 1 -- 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.
