Merge pull request #9540 from reitermarkus/unversioned-pkgs

Fix parsing version from `PackageInfo` files.

Library/Homebrew/Gemfile

@@ -5,6 +5,7 @@ source "https://rubygems.org"
 # installed gems (should all be require: false)
 gem "byebug", require: false
 gem "codecov", require: false
+gem "nokogiri", require: false
 gem "parallel_tests", require: false
 gem "ronn", require: false
 gem "rspec", require: false

Library/Homebrew/Gemfile.lock

@@ -169,6 +169,7 @@ DEPENDENCIES
   codecov
   concurrent-ruby
   mechanize
+  nokogiri
   parallel_tests
   patchelf
   plist

Library/Homebrew/unversioned_cask_checker.rb

@@ -57,10 +57,19 @@ module Homebrew
 
     sig { params(package_info_path: Pathname).returns(T.nilable(String)) }
     def self.version_from_package_info(package_info_path)
-      contents = package_info_path.read
+      Homebrew.install_bundler_gems!
+      require "nokogiri"
 
-      short_version = contents[/CFBundleShortVersionString="([^"]*)"/, 1].presence
-      version = contents[/CFBundleVersion="([^"]*)"/, 1].presence
+      xml = Nokogiri::XML(package_info_path.read)
+
+      bundle_id = xml.xpath("//pkg-info//bundle-version//bundle").first&.attr("id")
+      return unless bundle_id
+
+      bundle = xml.xpath("//pkg-info//bundle").find { |b| b["id"] == bundle_id }
+      return unless bundle
+
+      short_version = bundle["CFBundleShortVersionString"]
+      version = bundle["CFBundleVersion"]
 
       return decide_between_versions(short_version, version) if short_version && version
     end