Bug 810719

Summary: Circular build dependency in perl-SQL-Abstract-1.72-4.fc18
Product: [Fedora] Fedora Reporter: Paul Howarth <paul>
Component: perl-SQL-AbstractAssignee: Tom "spot" Callaway <tcallawa>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: perl-devel, tcallawa
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-12 18:17:35 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:
Attachments:
Description Flags
Spec change to split off perl-DBIx-Class-Storage-Debug-PrettyPrint package none

Description Paul Howarth 2012-04-08 16:23:41 UTC
Created attachment 576045 [details]
Spec change to split off perl-DBIx-Class-Storage-Debug-PrettyPrint package

perl-SQL-Abstract includes the DBIx::Class::Storage::Debug::PrettyPrint module, which requires DBIx::Class. This module is buildable without perl-DBIx-Class though, as the test suite does not exercise the PrettyPrint functionality.

perl-DBIx-Class requires SQL::Abstract but can't be bootstrapped because of the earlier-mentioned dependency.

A possible solution to this would be to split DBIx::Class::Storage::Debug::PrettyPrint off into its own subpackage, as per the attached patch.

Having done that, perl-DBIx-Class becomes bootable but fails the one test that requires DBIx::Class::Storage::Debug::PrettyPrint, so perl-DBIx-Class would need tweaking to BR: perl(DBIx::Class::Storage::Debug::PrettyPrint) except when bootstrapping, and to not run t/storage/dbic_pretty.t when bootstrapping.

That seems to me to be the cleanest way of resolving this cycle.

Comment 1 Tom "spot" Callaway 2012-04-12 18:17:35 UTC
Patch applied in rawhide, thanks.

Comment 2 Paul Howarth 2012-04-12 20:59:17 UTC
Note that perl-DBIx-Class also needs a complementary fix [BR:perl(DBIx::Class::Storage::Debug::PrettyPrint) except when
bootstrapping, and to not run t/storage/dbic_pretty.t when bootstrapping]. I guess I should raise another bug for that since that's owned by Iain Arnell.