Here is the next iteration of my patches. All the previous commits are now
just one single commit. I hope I corrected all the spelling errors in the
type and provider code.

I now also use the two arguments form of raise_error. That was a great
tip because it showed a few bugs in validate methods ;-)

The unit tests all pass on my system (Linux). But because of #5605 the type
is currently now useable.  The second commit introduces new integration
tests that will fail because of #5605.

About the name:
I don't like port either, but I dont like nss_*. I think that nss could be
an alternative provider that can also talk to a db or whatever is configured
in nsswitch. And that nss provider can then be used on any system that
supports nss.

Personally I prefer 'net_service' as the resource name.

-Stefan

-- 
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