Check that we have results before returning a value

Zach Leslie · Zach Leslie · commit 44728efa617d · 2015-09-07T19:24:27.000Z
Without this change, empty results yield a message like the following in
Facter output.

  undefined method `[]' for nil:NilClass

This work captures the results and check to them to ensure we return
a valid value.
diff --git a/lib/facter/python_version.rb b/lib/facter/python_version.rb
@@ -2,7 +2,10 @@
 
 def get_python_version(executable)
   if Facter::Util::Resolution.which(executable)
-    Facter::Util::Resolution.exec("#{executable} -V 2>&1").match(/^.*(\d+\.\d+\.\d+)$/)[1]
+    results = Facter::Util::Resolution.exec("#{executable} -V 2>&1").match(/^.*(\d+\.\d+\.\d+)$/)
+    if results
+      results[1]
+    end
   end
 end
 
