Re: [Puppet-dev] [PATCH/puppet 1/1] Proposed fix for #3013 (pipes, OSes, and ruby versions, Oh My!)

2010-01-10 Thread Paul Nasrat
2010/1/10 Markus Roberts mar...@reality.com: The recent release of 0.25.2 contains a new method of handling communication with execed processes to support SELinux and improve performance on other OSes which, unfortunately, has a number of shortcomings: Just had a quick scan through the patch

Re: [Puppet-dev] [PATCH/puppet 1/1] Proposed fix for #3013 (pipes, OSes, and ruby versions, Oh My!)

2010-01-10 Thread Markus Roberts
+options.on(--kill-me-withi SIGNAL,/USR[12]/) { |signal| $kill_me = signal } Looks like a typo in the options --kill-me-withi vs usage: +task = [File.dirname(__FILE__)+'/exec_test_helper','--kill-me-with','USR1','--called-by',$PID,'--repeat',n] Running with both

[Puppet-dev] [PATCH/puppet 1/1] Proposed fix for #3013 (pipes, OSes, and ruby versions, Oh My!)

2010-01-09 Thread Markus Roberts
The recent release of 0.25.2 contains a new method of handling communication with execed processes to support SELinux and improve performance on other OSes which, unfortunately, has a number of shortcomings: * It does not work on ruby 1.8.3 * It reintroduces one/possibly two bugs * It has no