Merge pull request #128 from fukayatsu/check-status-of-last-executed-child-process

Ensure $? is not nil

Author: julienXX

Ruby/lib/terminal-notifier.rb

@@ -51,7 +51,7 @@ def self.execute(verbose, options)
   # Raises if not supported on the current platform.
   def notify(message, options = {}, verbose = false)
     TerminalNotifier.execute(verbose, options.merge(:message => message))
-    $?.success?
+    $? && $?.success?
   end
   module_function :notify
 
@@ -61,7 +61,7 @@ def notify(message, options = {}, verbose = false)
   # If no ‘group’ ID is given, all notifications are removed.
   def remove(group = 'ALL', verbose = false)
     TerminalNotifier.execute(verbose, :remove => group)
-    $?.success?
+    $? && $?.success?
   end
   module_function :remove