Merge pull request #4889 from rolandwalker/plist_parse_updates

Plist parse updates
Homebrew · Jun 15, 2014 · edf513e · edf513e
2 parents 85cc3f9 + d61f165
commit edf513e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/cask/system_command.rb b/lib/cask/system_command.rb
@@ -63,11 +63,13 @@ def self._parse_plist(command, output)
         raise Plist::ParseError "Empty XML output"
       end
       xml
-    rescue Plist::ParseError
+    rescue Plist::ParseError => e
       raise CaskError.new(<<-ERRMSG)
 Error parsing plist output from command.
   command was:
   #{command.utf8_inspect}
+  error was:
+  #{e}
   output we attempted to parse:
   #{output}
         ERRMSG

diff --git a/lib/plist/parser.rb b/lib/plist/parser.rb
@@ -17,7 +17,7 @@ module Plist
 #  + Date elements are returned as DateTime objects.
 #  + Data elements are implemented as Tempfiles
 #
-# Plist::parse_xml will blow up if it encounters a data element.
+# Plist::parse_xml will blow up if it encounters a Date element.
 # If you encounter such an error, or if you have a Date element which
 # can't be parsed into a Time object, please send your plist file to
 # [email protected] so that I can implement the proper support.