Bug 810719 - Circular build dependency in perl-SQL-Abstract-1.72-4.fc18
Summary: Circular build dependency in perl-SQL-Abstract-1.72-4.fc18
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-SQL-Abstract
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-04-08 16:23 UTC by Paul Howarth
Modified: 2012-04-12 20:59 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-04-12 18:17:35 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Spec change to split off perl-DBIx-Class-Storage-Debug-PrettyPrint package (1.73 KB, application/x-download)
2012-04-08 16:23 UTC, Paul Howarth
no flags Details

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.


Note You need to log in before you can comment on or make changes to this bug.