Bug 810370

Summary: Circular build dependency in perl-DBI-1.618-2.fc18
Product: [Fedora] Fedora Reporter: Paul Howarth <paul>
Component: perl-DBIAssignee: Marcela Mašláňová <mmaslano>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: mmaslano, perl-devel, ppisar, psabata
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-04-06 12:47:22 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Paul Howarth 2012-04-05 19:07:00 UTC
perl-DBI has a buildreq of perl(SQL::Statement) >= 1.28, which is used in the test suite if available. However, SQL::Statement itself requires DBI, so this creates a circular build dependency, which will be a problem when we come to build everything for perl 5.16. Suggested fix is to not buildrequire SQL::Statement when bootstrapping:

diff --git a/perl-DBI.spec b/perl-DBI.spec
index 0dc2f0c..3039553 100644
--- a/perl-DBI.spec
+++ b/perl-DBI.spec
@@ -38,8 +38,10 @@ BuildRequires:  perl(RPC::PlClient) >= 0.2000
 # RPC::PlServer is optional
 BuildRequires:  perl(RPC::PlServer)
 BuildRequires:  perl(Scalar::Util)
-# SQL::Statement is optional
+# SQL::Statement is optional, and it requires DBI
+%if 0%{!?perl_bootstrap:1}
 BuildRequires:  perl(SQL::Statement) >= 1.28
+%endif
 BuildRequires:  perl(Storable)
 BuildRequires:  perl(Symbol)
 BuildRequires:  perl(threads)