On May 23, 2011, at 11:53 AM, Jacob Helwig wrote:

> So, I'm working on #7188 and thought "hey, it would be great if we had
> built-in checks that these required parameters were actually specified".
> Especially since the current mount type claims that fstype is required,
> yet does nothing (other than break things after a successful run) if
> it's not specified (tested using 2.7.x on Solaris).
> 
> Unfortunately, it looks like calling isrequired from within the block of
> newproperty doesn't actually do this (largely because it doesn't look
> like anything is actually checking required?).
> 
> Has setting isrequired ever worked, or are isrequired and required?
> Left-overs from an aborted attempt?
> 
> Is there an implemented way of doing 'this parameter is required; fail
> at compile time'?  Hopefully without having to do a defaultto of a bogus
> value, and having a validation failure against that.

It looks like I just never finished the implementation of that.  Weird.

-- 
There is only one success - to be able to spend your life in your own
way.        -- Christopher Morley
---------------------------------------------------------------------
Luke Kanies  -|-   http://puppetlabs.com   -|-   http://about.me/lak




-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" 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-dev?hl=en.

Reply via email to