On Wed, Aug 28, 2013 at 12:36 PM, Adrien Thebo <[email protected]>wrote:
> On Wed, Aug 28, 2013 at 12:33 PM, Brian Warsing <[email protected]>wrote: > >> Can you clarify: PuppetLabs patches the CFPropertyList gem before vending >> it? >> > > We have a pull request targeted at Facter that handles string encoding > against CFPropertyList (https://github.com/puppetlabs/facter/pull/513); > for cases like this if we're going to accept that change _and_ get it > upstream, we would probably have to handle the submission against upstream > ourselves. > Wait, sorry, the question was if we're currently patching CFPropertyList. The answer is yes, we have to change the namespacing to avoid potential collisions, https://github.com/puppetlabs/facter/blob/master/lib/facter/util/cfpropertylist/lib/rbCFPropertyList.rb#L45 for example. -- Adrien Thebo | Puppet Labs -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-dev. For more options, visit https://groups.google.com/groups/opt_out.
