On Thu, 12 May 2011 18:20:11 -0700, Michael Kincaid wrote: > > Signed-off-by: Michael Kincaid <[email protected]> > --- > Local-branch: ticket/next/6515 > lib/facter/processor.rb | 6 ++++++ > 1 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/lib/facter/processor.rb b/lib/facter/processor.rb > index df4e9af..44bb3e7 100644 > --- a/lib/facter/processor.rb > +++ b/lib/facter/processor.rb > @@ -56,6 +56,12 @@ Facter.add("ProcessorCount") do > end > end > > +Facter.add("Processor") do > + confine :kernel => :openbsd > + setcode do > + Facter::Util::Resolution.exec("uname -p") > + end > +end > > Facter.add("ProcessorCount") do > confine :kernel => :openbsd > -- > 1.7.4.1 >
Michael, So, I'm not quite sure why this is being done. I didn't see a ticket related to it after a quick search (and I certainly didn't get enough info from the commit message to justify the change ;). What's the motivation for this? Is there a pre-existing ticket it should be associated with? Got a test to go along with this? ;) I don't doubt that this is a change that we should be doing, but I'd love to know why. -- Jacob Helwig
signature.asc
Description: Digital signature
