Issue #5079 has been updated by Matt Robinson. Status changed from Code Insufficient to Ready for Checkin Assignee deleted (Sandor Szücs)
After talking with Sandor on the dev list I'm going to merge in my rework of his refactor patch. ---------------------------------------- Refactor #5079: MCX content provider https://projects.puppetlabs.com/issues/5079 Author: Sandor Szücs Status: Ready for Checkin Priority: Normal Assignee: Category: OSX Target version: Affected Puppet version: Branch: I refactored the MCX content provider to enhance code readability. This is the first step to understand the current implementation. I would like to add a best practice dslocal solution pointed out by the authors of "Enterprise Mac Managed Preferences". They wrote that you should not use /Local/Default as DB, because it is also used as local cache. They further said that you should add an own DB to circumvent ping-pong overwrites between central managed MCX and dslocal managed MCX. The patch includes a small change within the spec file, because the error handling should be more conform to other puppet providers. Instead to raise MCXContentProviderException I choose to raise Puppet::Error, because macauthorization provider does the same. I hope that is the right Error handling. -- 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.
