lib/rubyipmi.rb in rubyipmi-0.9.1 vs lib/rubyipmi.rb in rubyipmi-0.9.2

@@ -152,12 +152,12 @@
       when "freeipmi"
         cmdpath = locate_command('ipmipower')
       when "ipmitool"
         cmdpath = locate_command('ipmitool')
       else
-        logger.error("Invalid BMC provider type") if logger
-        raise "Invalid BMC provider type"
+        logger.error("Invalid BMC provider type #{provider}") if logger
+        false
     end
     # return false if command was not found
     return ! cmdpath.nil?
   end
 
@@ -165,13 +165,13 @@
     ["freeipmi", "ipmitool"]
   end
 
   # returns true if any of the providers are installed
   def self.provider_installed?
-    providers_installed?.length > 0
+    providers_installed.length > 0
   end
 
-  def self.providers_installed?
+  def self.providers_installed
     available = []
     providers.each do |prov|
       if is_provider_installed?(prov)
         available << prov
       end