Bug 766440

Summary: yum-builddep doesn't check the return code from doUtilBuildTransaction ... mock failures
Product: Red Hat Enterprise Linux 6 Reporter: Clint Savage <herlo1>
Component: yum-utilsAssignee: James Antill <james.antill>
Status: CLOSED CURRENTRELEASE QA Contact: BaseOS QE Security Team <qe-baseos-security>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.0CC: bochecha, ebenes, ksrot, zpavlas
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-03-06 11:24:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Clint Savage 2011-12-12 03:44:07 UTC 
Description of problem:

Using koji and mock to build PyQt4-4.6.2-8. i686 build fails to pull in kernel dependency properly. x86_64 build appears to work properly. This appears to be specific to yum for el6

Version-Release number of selected component (if applicable):

$ rpm -q yum
yum-3.2.29-22.el6.noarch

How reproducible:

100% reproducible on el6.0

Steps to Reproduce:
1. koji build <dist> git://<git_url>/<pkg>.git#HEAD launches a koji build, which in turn launches mock
2. SRPM builds properly and launches arch-specific builds
3. i686 build fails with depsolve errors described below
  
Actual results:

ERROR with rpm_check_debug vs depsolve:
DEBUG util.py:257:  kernel >= 2.6.29.1-52.fc11 is needed by libdrm-2.4.20-2.gl6.i686

Detailed information can be found @ 
http://koji.gooselinux.org/koji/taskinfo?taskID=32099

Expected results:

Successful depsolve.

Additional info:
Comment 2 James Antill 2011-12-20 20:12:47 UTC 
 This seems to work here, my guess is that it's this change to yum-builddep:

--- yum-utils-1.1.30/yum-builddep.py	2011-09-01 14:58:03.099739374 -0400
+++ /usr/bin/yum-builddep	2011-08-13 10:23:17.000000000 -0400
@@ -99,7 +98,9 @@
             sys.exit(1)
 
         if hasattr(self, 'doUtilBuildTransaction'):
-            self.doUtilBuildTransaction()
+            errc = self.doUtilBuildTransaction()
+            if errc:
+                sys.exit(errc)
         else:
             try:
                 self.buildTransaction()
Comment 8 Karel Srot 2012-03-06 08:54:46 UTC 
Hi Clint,
this should be already fixed in yum-utils-1.1.30-10.el6 from RHEL-6.2.
Can you confirm that?
Comment 9 Zdeněk Pavlas 2012-03-06 11:24:08 UTC 
Checked that BZ-709043-check-doutilbuildtransaction.patch has fixed all 5 callers of doUtilBuildTransaction.