From e9807473612a6961d566e353b95a67f1cc72dccb Mon Sep 17 00:00:00 2001
From: donghee yang <donghee.yang@samsung.com>
Date: Tue, 4 Sep 2012 07:59:02 +0900
Subject: [PATCH] [Title] Removed reverse build's inefficiency on remote build
 server

---
 package/pkginfo.manifest                     |  2 +-
 src/build_server/BuildJob.rb                 | 14 ++++++++++++--
 src/build_server/RemoteBuilder.rb            |  4 ++--
 src/build_server/SocketJobRequestListener.rb |  6 +++++-
 4 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/package/pkginfo.manifest b/package/pkginfo.manifest
index 9ff6e77..dfe11b3 100644
--- a/package/pkginfo.manifest
+++ b/package/pkginfo.manifest
@@ -1,5 +1,5 @@
 Source : dibs
-Version :0.99.33
+Version :0.99.34
 Maintainer : taejun ha<taejun.ha@samsung.com>, jiil hyoun <jiil.hyoun@samsung.com>, donghyuk yang <donghyouk.yang@samsung.com>, donghee yang <donghee.yang@samsung.com>, sungmin kim <dev.sungmin.kim@samsung.com
 
 Package : dibs
diff --git a/src/build_server/BuildJob.rb b/src/build_server/BuildJob.rb
index 702014e..7f19300 100644
--- a/src/build_server/BuildJob.rb
+++ b/src/build_server/BuildJob.rb
@@ -147,6 +147,11 @@ class BuildJob
 	end
 
 
+	def is_rev_build_check_job()
+		return @is_rev_build_check_job
+	end
+
+
 	def set_remote_job(server)
 		@is_remote_job = true
 		@remote_server=server
@@ -1092,8 +1097,13 @@ class BuildJob
 		@log.info( "Copying log to #{outgoing_dir}", Log::LV_USER)
 		file = "#{@source_path}/../log"
 		FileUtils.copy_file(file, "#{outgoing_dir}/remote_log")
-
-		return copy_result_files( outgoing_dir )
+	
+		# copy result files, if not reverse build	
+		if not @is_rev_build_check_job then
+			return copy_result_files( outgoing_dir )
+		else
+			return true
+		end
 	end
 
 
diff --git a/src/build_server/RemoteBuilder.rb b/src/build_server/RemoteBuilder.rb
index f42a318..76e3191 100644
--- a/src/build_server/RemoteBuilder.rb
+++ b/src/build_server/RemoteBuilder.rb
@@ -162,8 +162,8 @@ class RemoteBuilder
 					break
                 end
 
-				# gather result files
-				if l =~ /Creating package file \.\.\. (.*)/ then
+				# gather result files if not reverse build
+				if not is_rev_build and l =~ /Creating package file \.\.\. (.*)/ then
 					file_name = $1
 					result_files.push file_name
 				end
diff --git a/src/build_server/SocketJobRequestListener.rb b/src/build_server/SocketJobRequestListener.rb
index aaf4ca3..32022b0 100644
--- a/src/build_server/SocketJobRequestListener.rb
+++ b/src/build_server/SocketJobRequestListener.rb
@@ -268,7 +268,11 @@ class SocketJobRequestListener
 		end	
 
 		# add to job queue
-		@parent_server.jobmgr.add_job( new_job )
+		if new_job.is_rev_build_check_job() then
+			@parent_server.jobmgr.add_reverse_build_job( new_job )
+		else
+			@parent_server.jobmgr.add_job( new_job )
+		end
 	end
 
 
-- 
2.34.1