#298: added checking by release

app/models/build_list.rb

@@ -283,13 +283,20 @@ class BuildList < ActiveRecord::Base
   def has_new_packages?
     if last_bl = last_published.joins(:source_packages).where(:build_list_packages => {:actual => true}).last
       source_packages.each do |nsp|
+        # priority: EPOCH => VERSION => RELEASE
+        # TODO: EPOCH
         sp = last_bl.source_packages.find{ |sp| nsp.name == sp.name }
         return true unless sp
         sp_version  = sp.version.split(/\D/).map(&:to_i)
-        nsp.version.split(/\D/).map(&:to_i).each_with_index do |nv, index|
+        nsp_version = nsp.version.split(/\D/).map(&:to_i)
+        if nsp_version == sp_version
+          return true if nsp.release.to_i > sp.release.to_i
+        else
+          nsp_version.each_with_index do |nv, index|
             return true if nv > sp_version[index].to_i
           end
         end
+      end
     else
       return true # no published packages
     end

spec/models/build_list_spec.rb

@@ -188,6 +188,11 @@ describe BuildList do
       build_list.can_auto_publish?.should be_true
     end
 
+    it 'ensures that return true if release of published package <' do
+      published_build_list_package.update_attributes(:release => 5)
+      build_list.can_auto_publish?.should be_true
+    end
+
   end
 
 end