Hi all,

I have made the yumrepo type ensurable, but judging from the comments I see 
in the pull request, it would be great to truly refactor this and separate 
the provider logic from the provider.
Therefore, I have a few questions:

Why is IniProperty in the yumrepo type? What should I do with it? It is 
required by all properties.
Not sure what to do with instances, inifile, store, flush.. I know I have 
to move them to the provider but I can't figure out an elegant way to do it 
without overloading the provider. What about the class << self I can find 
on the type?

Here's the pull request, it works (makes my yumrepos ensurable) but the 
type/provider separation thing has been bugging me for a while

https://github.com/puppetlabs/puppet/pull/1236

Best,

-- 
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].
Visit this group at http://groups.google.com/group/puppet-dev?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to