Issue #4258 has been updated by Dominic Cleal.
Thanks for the feedback Matt. I've moved the new pkgutil.conf test inside a "healthcheck" method, which now contains the tests that were in the extended method. From what I can tell, they were never being run to begin with (having been copied from blastwave.rb)! This method is then called from the prefetch and is stubbed out in the spec. The merge conflict looks like a problem on Juerg's branch as he's fixed the same issue as me, then merged in the same commit from my branch. The above commits, along with Juerg's to add the pkgutil.conf check have been pushed to my own branch (https://github.com/domcleal/puppet/tree/tickets/master/4258-dev) and I've verified it merges into next. ---------------------------------------- Feature #4258: Add pkgutil provider https://projects.puppetlabs.com/issues/4258 Author: James Turnbull Status: Ready For Testing Priority: Normal Assignee: Matt Robinson Category: Solaris Target version: Statler Affected Puppet version: 0.25.5 Keywords: communitypatch Branch: https://github.com/domcleal/puppet/tree/tickets/master/4258-dev -- 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.
