Re: [Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-06 Thread Paul Nasrat
On 6 August 2010 00:35, Rein Henrichs r...@puppetlabs.com wrote: * Per #4466, Ruby has trouble reading files in /proc [1]. The  alternative is to use `bin/cat`. * Also refactored methods to explicitly redirect standard error to  /dev/null for *nix and BSD system calls. +1 Random thought for

Re: [Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-06 Thread Paul Berry
On Fri, Aug 6, 2010 at 2:23 AM, Paul Nasrat pnas...@gmail.com wrote: Random thought for 1.6 or later we could have a ProcFile and Sysctl specific class (and dmidecode) to encapsulate the complexities of different platforms/versions so it's easy to do stuff and provides a handy way to prevent

Re: [Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-06 Thread Paul Nasrat
On 6 August 2010 17:55, Paul Berry p...@puppetlabs.com wrote: On Fri, Aug 6, 2010 at 2:23 AM, Paul Nasrat pnas...@gmail.com wrote: Random thought for 1.6 or later we could have a ProcFile and Sysctl specific class (and dmidecode) to encapsulate the complexities of different platforms/versions

Re: [Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-06 Thread Rein Henrichs
Excerpts from Paul Berry's message of Fri Aug 06 09:55:37 -0700 2010: On Fri, Aug 6, 2010 at 2:23 AM, Paul Nasrat pnas...@gmail.com wrote: Random thought for 1.6 or later we could have a ProcFile and Sysctl specific class (and dmidecode) to encapsulate the complexities of different

Re: [Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-06 Thread R.I.Pienaar
- Rein Henrichs r...@puppetlabs.com wrote: Excerpts from Paul Berry's message of Fri Aug 06 09:55:37 -0700 2010: On Fri, Aug 6, 2010 at 2:23 AM, Paul Nasrat pnas...@gmail.com wrote: Random thought for 1.6 or later we could have a ProcFile and Sysctl specific class (and

Re: [Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-06 Thread Markus Roberts
+1 or, as they used to say on AOL, Me Too! On Fri, Aug 6, 2010 at 10:12 AM, R.I.Pienaar r...@devco.net wrote: - Rein Henrichs r...@puppetlabs.com wrote: Excerpts from Paul Berry's message of Fri Aug 06 09:55:37 -0700 2010: On Fri, Aug 6, 2010 at 2:23 AM, Paul Nasrat pnas...@gmail.com

[Puppet-dev] [PATCH/facter 2/2] [#2330] Update uptime calculation to use /bin/cat

2010-08-05 Thread Rein Henrichs
* Per #4466, Ruby has trouble reading files in /proc [1]. The alternative is to use `bin/cat`. * Also refactored methods to explicitly redirect standard error to /dev/null for *nix and BSD system calls. [1] http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-talk/155745 Signed-off-by: