Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Circular build dependency in perl-Devel-PartialDump-0.15-3.fc18

https://bugzilla.redhat.com/show_bug.cgi?id=810532

           Summary: Circular build dependency in
                    perl-Devel-PartialDump-0.15-3.fc18
           Product: Fedora
           Version: rawhide
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: unspecified
          Priority: unspecified
         Component: perl-Devel-PartialDump
        AssignedTo: iarn...@gmail.com
        ReportedBy: p...@city-fan.org
         QAContact: extras...@fedoraproject.org
                CC: iarn...@gmail.com, fedora-perl-devel-l...@redhat.com
    Classification: Fedora
      Story Points: ---
              Type: Bug
        Regression: ---
        Mount Type: ---
     Documentation: ---


perl-Devel-PartialDump buildrequires and requires Moose, but Moose itself has a
runtime dependency on perl(Devel::PartialDump) and so cannot be used to
bootstrap perl-Devel-PartialDump. Suggested fix is to avoid buildreq of Moose
and skip the test suite when bootstrapping:

diff --git a/perl-Devel-PartialDump.spec b/perl-Devel-PartialDump.spec
index 18c3912..d27d12d 100644
--- a/perl-Devel-PartialDump.spec
+++ b/perl-Devel-PartialDump.spec
@@ -9,7 +9,13 @@ URL:            http://search.cpan.org/dist/Devel-PartialDump/
 Source0:       
http://www.cpan.org/authors/id/F/FL/FLORA/Devel-PartialDump-%{version}.tar.gz
 BuildArch:      noarch
 BuildRequires:  perl(ExtUtils::MakeMaker)
+# Moose has a manual dependency on Devel::PartialDump for enhanced reporting
+# Devel::PartialDump itself requires Moose, so when bootstrapping we do a
blind
+# build of Devel::PartialDump without running the test suite, then build
Moose,
+# then rebuild Devel::PartialDump with Moose, running the test suite
+%if 0%{!?perl_bootstrap:1}
 BuildRequires:  perl(Moose)
+%endif
 BuildRequires:  perl(namespace::clean) >= 0.20
 BuildRequires:  perl(Sub::Exporter)
 BuildRequires:  perl(Test::use::ok)
@@ -38,7 +44,9 @@ find $RPM_BUILD_ROOT -depth -type d -exec rmdir {}
2>/dev/null \;
 %{_fixperms} $RPM_BUILD_ROOT/*

 %check
+%if 0%{!?perl_bootstrap:1}
 make test
+%endif

 %files
 %doc Changes


All packages using %perl_bootstrap should of course be rebuilt without
%perl_bootstrap set once the bootstrap build is complete, so we find any FTBFS
issues revealed by skipped tests.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/perl-devel

Reply via email to