Re: [Puppet-dev] facter and indent spacing

2011-09-19 Thread Luke Kanies
I agree. Maybe it's worth a merge-a-thon to get as many merged as possible in a fixed amount of time, then just draw the line? -- Luke Kanies | +1-615-594-8199 | http:/about.me/lak On Sep 19, 2011, at 3:22 PM, Adrien Thebo wrote: I'm of two minds of this. On one hand, it would be terribly nice

[Puppet-dev] (puppetlabs/facter) Closed pull request: (#2747) Fix detection of xen0 vs xenu in Xen 3.2.

2011-09-19 Thread wearetherobots
Greetings! On Mon Sep 19 22:56:13 UTC 2011 pull request #54 was closed. (#2747) Fix detection of xen0 vs xenu in Xen 3.2. requested by (mkincaid) The pull request was merged by: adrienthebo Thanks! The Pull Request Bot -- You received this message becau

Re: [Puppet-dev] facter and indent spacing

2011-09-19 Thread Adrien Thebo
I'm of two minds of this. On one hand, it would be terribly nice to get as many patches merged as possible before we do a whitespace change just so that we don't have to deal with any weird conflicts. On the other hand, as you said git can probably handle the whitespace change, and I would like to

[Puppet-dev] (puppetlabs/puppet) New pull request: Ticket/2.7.x/8341 prevent duplicate loading of facts

2011-09-19 Thread wearetherobots
Greetings! Please review the pull request #127: Ticket/2.7.x/8341 prevent duplicate loading of facts opened by (adrienthebo) Some more information about the pull request: Opened: Mon Sep 19 21:59:10 UTC 2011 Based on: puppetlabs:2.7.x (18afb3a5262b1447c6bb2

[Puppet-dev] Open Source Team iteration & planning summary 2011-09-19

2011-09-19 Thread Jacob Helwig
Getting & setting permissions on Windows now works (#9186). For full details on how this works, please see the commit that introduces the functionality[1]. Patch provided by Josh Cooper. This has been merged into the 2.7.x branch. Pluginsync will now only load Ruby files (#4135). Pluginsync wil

[Puppet-dev] (puppetlabs/facter) Closed pull request: (#9593) Require rubygems to handle json output for ruby 1.8.7.

2011-09-19 Thread wearetherobots
Greetings! On Mon Sep 19 16:16:52 UTC 2011 pull request #60 was closed. (#9593) Require rubygems to handle json output for ruby 1.8.7. requested by (kbarber) Thanks! The Pull Request Bot -- You received this message because you are subscribed to the Google G

Re: [Puppet-dev] Re: Add the example 'log' listener

2011-09-19 Thread Josh Cooper
Hi Brice, Thanks for resubmitting. We are wrapping up Windows support this week, and will be back reviewing patches soon. Thanks again, Josh -- Join us for PuppetConf, September 22nd and 23rd in Portland, OR: http://bit.ly/puppetconfsig On Sun, Sep 18, 2011 at 6:55 AM, Brice Figureau < brice-pu

[Puppet-dev] (puppetlabs/facter) New pull request: (#9593) Require rubygems to handle json output for ruby 1.8.7.

2011-09-19 Thread wearetherobots
Greetings! Please review the pull request #60: (#9593) Require rubygems to handle json output for ruby 1.8.7. opened by (kbarber) Some more information about the pull request: Opened: Mon Sep 19 14:12:10 UTC 2011 Based on: puppetlabs:master (ef0a9aa3e36b371

[Puppet-dev] facter and indent spacing

2011-09-19 Thread Ken Barber
Hi, So I raised a ticket for fixing mixed whitespace in facter: http://projects.puppetlabs.com/issues/9555 And I've got a preliminary patch in a topic branch just to see how bad the problem is. The reason I've raised this is that people making changes across multiple files at the moment are find