From a0a153611521459f939377f45fbe06e47d56e7b7 Mon Sep 17 00:00:00 2001
From: donghee yang <donghee.yang@samsung.com>
Date: Fri, 12 Apr 2013 09:52:45 +0900
Subject: [PATCH] [Title] Fixed a bug of job assignment

---
 package/changelog            | 3 +++
 package/pkginfo.manifest     | 2 +-
 src/build_server/BuildJob.rb | 8 +++++---
 3 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/package/changelog b/package/changelog
index 4ec4b9e..f8449a7 100644
--- a/package/changelog
+++ b/package/changelog
@@ -1,3 +1,6 @@
+* 2.1.24
+- Fixed a bug that job assignment to child server is wrong
+== donghee yang <donghee.yang@samsung.com> 2013-04-12
 * 2.1.23
 - Modified to make build-server work when max-job is zero
 == donghee yang <donghee.yang@samsung.com> 2013-04-12
diff --git a/package/pkginfo.manifest b/package/pkginfo.manifest
index bc6f0b3..bc39ff5 100644
--- a/package/pkginfo.manifest
+++ b/package/pkginfo.manifest
@@ -1,5 +1,5 @@
 Source : dibs
-Version :2.1.23
+Version :2.1.24
 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 7985461..b9c9dc9 100644
--- a/src/build_server/BuildJob.rb
+++ b/src/build_server/BuildJob.rb
@@ -237,11 +237,13 @@ class BuildJob < CommonJob
 		if @pkginfo.nil? then return false end
 
 		@pkginfo.packages.each do |pkg|
-			if pkg.os_list.include? @os and pkg.build_host_os.include? host_os then
-				return true
+			if not pkg.os_list.include? @os then next end
+			if not pkg.build_host_os.include? host_os then
+				return false
 			end
 		end
-		return false
+
+		return true
 	end
 
 
-- 
2.34.1