From: jiil.hyoun <jiil.hyoun@samsung.com>
Date: Tue, 23 Oct 2012 01:53:18 +0000 (+0900)
Subject: [Title] do not check change log when reverse job
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d68cccba4b9165ddbaf9b74b8f1404aab7ffc1b1;p=sdk%2Ftools%2Fsdk-build.git

[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 b9b1e7b..8d1252c 100644
--- 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)