Fix the version prefix check to understand RPM epochs.

poise · Apr 18, 2018 · c5db46d · c5db46d
1 parent 71cb3a4
commit c5db46d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/poise_languages/system/resource.rb b/lib/poise_languages/system/resource.rb
@@ -233,8 +233,8 @@ def patch_load_current_resource!(provider, version)
               each_package do |package_name, new_version, current_version, candidate_version|
                 # In Chef 12.14+, candidate_version is a Chef::Decorator::Lazy object
                 # so we need the nil? check to see if the object being proxied is
-                # nil (i.e. there is no version).
-                unless candidate_version && (!candidate_version.nil?) && (!candidate_version.empty?) && candidate_version.start_with?(version)
+                # nil (i.e. there is no version). The `\d+:` is for RPM epoch prefixes.
+                unless candidate_version && (!candidate_version.nil?) && (!candidate_version.empty?) && candidate_version =~ /^(\d+:)?#{Regexp.escape(version)}/
                   # Don't display a wonky error message if there is no candidate.
                   candidate_label = if candidate_version && (!candidate_version.nil?) && (!candidate_version.empty?)
                     candidate_version