Bug 810865 - Circular build dependency in perl-Dancer-1.3093-1.fc18
Summary: Circular build dependency in perl-Dancer-1.3093-1.fc18
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-Dancer
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Marcela Mašláňová
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-04-09 12:53 UTC by Paul Howarth
Modified: 2012-04-26 09:56 UTC (History)
3 users (show)

Fixed In Version: perl-Dancer-1.3095-1.fc18
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-04-26 09:22:08 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Paul Howarth 2012-04-09 12:53:13 UTC
perl-Dancer has an optional test dependency of perl(Dancer::Session::Cookie) >= 0.14, which pulls in perl-Dancer-Session-Cookie. However, that requires perl-Dancer itself and thus introduces a circular dependency. Suggested fix - wrap that buildreq with a %perl_bootstrap conditional:

diff --git a/perl-Dancer.spec b/perl-Dancer.spec
index c7ee3be..04084a9 100644
--- a/perl-Dancer.spec
+++ b/perl-Dancer.spec
@@ -62,7 +62,10 @@ BuildRequires:  perl(Test::Builder)
 BuildRequires:  perl(URI::Escape)
 # Optional tests:
 BuildRequires:  perl(HTTP::Parser::XS)
+# Dancer::Session::Cookie needs Dancer itself
+%if 0%{!?perl_bootstrap:1}
 BuildRequires:  perl(Dancer::Session::Cookie) >= 0.14
+%endif
 Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
 Requires:       perl(HTTP::Body) >= 1.07
 Requires:       perl(HTTP::Server::Simple::PSGI) >= 0.11

I tested this on F-17 as mod_perl is pulled in as a buildreq and it's currently uninstallable in Rawhide.

Comment 1 Marcela Mašláňová 2012-04-10 09:21:04 UTC
Thanks, it's test BR only.

Comment 2 Marcela Mašláňová 2012-04-10 10:19:27 UTC
Waiting for fix of httpd-mmn.

Comment 3 Marcela Mašláňová 2012-04-26 09:22:08 UTC
I guess this is fixed now.

Comment 4 Paul Howarth 2012-04-26 09:56:49 UTC
Yes, perl-Dancer-1.3095-1.fc18 has the fix, thanks. I just built it since the original attempt failed due to mod_perl.


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