[Title] do not check change log when reverse job

[Type] Bugfix
[Module] Toolchain /
[Priority] Major
[Jira#]
[Redmine#]
[Problem]
[Cause]
[Solution]
[TestCase]

Change-Id: I65d83b229b3be66fd262e8937b1bdf5f3baa37ab

diff --git a/src/build_server/GitBuildJob.rb b/src/build_server/GitBuildJob.rb
index b9b1e7be05af374d3d4f4134df3592c90b303d1a..8d1252c8868d6996ac4b3f0ba0beab68da9d4837 100644 (file)
--- a/src/build_server/GitBuildJob.rb
+++ b/src/build_server/GitBuildJob.rb
@@ -156,29 +156,30 @@ class GitBuildJob < BuildJob
                        return false
                end
 
-               #set up change log
-               change_log = {}
-               begin
-                       change_log = Parser.read_changelog "#{@source_path}/package/changelog" if File.exist? "#{@source_path}/package/changelog"
-               rescue => e
-                       @log.error( e.message, Log::LV_USER)
-                       return false
-               end
-
-               if not change_log.empty? and @pkginfo.packages[0].change_log.empty? then
-                       @pkginfo.packages.each {|pkg| pkg.change_log = change_log}
-               end
-
-               if @server.changelog_check and not @pkginfo.packages[0].does_change_exist? then
-                       @log.error( "change log not found", Log::LV_USER )
-                       return false
-               end
-
                # set up pkgsvr_client
                @pkgsvr_client =  Client.new(@pkgsvr_url, @job_working_dir, @log)
 
                # checking version if not reverse-build job or not internal-job
                if not @is_rev_build_check_job and not @is_internal_job  then
+
+                       #check change log
+                       change_log = {}
+                       begin
+                               change_log = Parser.read_changelog "#{@source_path}/package/changelog" if File.exist? "#{@source_path}/package/changelog"
+                       rescue => e
+                               @log.error( e.message, Log::LV_USER)
+                               return false
+                       end
+
+                       if not change_log.empty? and @pkginfo.packages[0].change_log.empty? then
+                       @pkginfo.packages.each {|pkg| pkg.change_log = change_log}
+                       end
+
+                       if @server.changelog_check and not @pkginfo.packages[0].does_change_exist? then
+                               @log.error( "change log not found", Log::LV_USER )
+                               return false
+                       end
+
                        # check availabiltiy
                        if not @server.check_job_availability( self ) then
                                @log.error( "No servers that are able to build your packages.", Log::LV_USER)