Issue #13952 has been updated by Daniel Pittman.

Category changed from inventory service to language


----------------------------------------
Bug #13952: 'false' matches . but not false or default.
https://projects.puppetlabs.com/issues/13952#change-61078

Author: Jos Boumans
Status: Needs Decision
Priority: Normal
Assignee: Pieter van de Bruggen
Category: language
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.

Reply via email to