Simplify confine statement

Author: ekohl

lib/puppet/provider/service/init.rb

@@ -20,12 +20,9 @@ def self.defpath
   end
 
   # Debian and Ubuntu should use the Debian provider.
+  confine :false => ['Debian', 'Ubuntu'].include?(Puppet.runtime[:facter].value('os.name'))
   # RedHat systems should use the RedHat provider.
-  confine :true => begin
-      os = Puppet.runtime[:facter].value('os.name')
-      family = Puppet.runtime[:facter].value('os.family')
-      !(os == 'Debian' || os == 'Ubuntu' || family == 'RedHat')
-  end
+  confine :false => Puppet.runtime[:facter].value('os.family') == 'RedHat'
 
   # We can't confine this here, because the init path can be overridden.
   #confine :exists => defpath