Bug 810721

Summary: Circular build dependencies: perl-Test-WWW-Mechanize-Catalyst <-> perl-Catalyst-Plugin-Session-State-Cookie
Product: [Fedora] Fedora Reporter: Paul Howarth <paul>
Component: perl-Catalyst-Plugin-Session-State-CookieAssignee: Iain Arnell <iarnell>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: iarnell, mmaslano, perl-devel, ppisar
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-04-18 23:09:06 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-08 17:04:42 UTC
These two packages buildrequire each other. They're both optional test dependencies so it's a toss-up as to which one to break the cycle in. I tested both in F-17 as the packages require mod_perl, which is currently uninstallable in Rawhide. The perl-Catalyst-Plugin-Session-State-Cookie test suite skips a bit more gracefully so I'd tweak that one personally:

diff --git a/perl-Test-WWW-Mechanize-Catalyst.spec b/perl-Test-WWW-Mechanize-Catalyst.spec
index 5bd8df9..6aa12aa 100644
--- a/perl-Test-WWW-Mechanize-Catalyst.spec
+++ b/perl-Test-WWW-Mechanize-Catalyst.spec
@@ -10,7 +10,11 @@ Requires:       perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $versi
 BuildArch:      noarch
 
 BuildRequires:  perl(Catalyst) >= 5.00
+# Catalyst::Plugin::Session::State::Cookie and Test::WWW::Mechanize::Catalyst
+# use each other in their test suites
+%if 0%{!?perl_bootstrap:1}
 BuildRequires:  perl(Catalyst::Plugin::Session::State::Cookie)
+%endif
 BuildRequires:  perl(Catalyst::Plugin::Session::Store::Dummy)
 BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.42
 BuildRequires:  perl(LWP) >= 5.816


OR


diff --git a/perl-Catalyst-Plugin-Session-State-Cookie.spec b/perl-Catalyst-Plugin-Session-State-Cookie.spec
index 5ee95f4..9b0b929 100644
--- a/perl-Catalyst-Plugin-Session-State-Cookie.spec
+++ b/perl-Catalyst-Plugin-Session-State-Cookie.spec
@@ -20,7 +20,11 @@ BuildRequires:  perl(namespace::autoclean)
 BuildRequires:  perl(Test::MockObject) >= 1.01
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(Test::Pod)
+# Test::WWW::Mechanize::Catalyst and Catalyst::Plugin::Session::State::Cookie
+# use each other in their test suites
+%if 0%{!?perl_bootstrap:1}
 BuildRequires:  perl(Test::WWW::Mechanize::Catalyst)
+%endif
 
 Requires:       perl(Catalyst) >= 5.80005
 Requires:       perl(Catalyst::Plugin::Session) >= 0.27

Comment 1 Fedora Update System 2012-04-11 11:33:53 UTC
perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17

Comment 2 Iain Arnell 2012-04-11 11:35:22 UTC
Patch applied to Test-WWW-Mechanize-Catalyst because it was already due for an update to latest upstream version. Committed to rawhide and f17; built only for f17 since mod_perl is still broken.

Comment 3 Fedora Update System 2012-04-12 01:10:53 UTC
Package perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-5725/perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17
then log in and leave karma (feedback).

Comment 4 Fedora Update System 2012-04-18 23:09:06 UTC
perl-Test-WWW-Mechanize-Catalyst-0.57-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.