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

            Bug ID: 1437796
           Summary: Circular build dependency on perl-Type-Tiny
           Product: Fedora
           Version: rawhide
         Component: perl-Type-Tie
          Assignee: rc040...@freenet.de
          Reporter: ppi...@redhat.com
        QA Contact: extras...@fedoraproject.org
                CC: extras...@fedoraproject.org,
                    perl-devel@lists.fedoraproject.org,
                    rc040...@freenet.de



+++ This bug was initially created as a clone of Bug #1433344 +++

perl-Type-Tiny cannot be bootstrapped because it has three circular
dependencies on:

(1) perl-Type-Tie: perl-Type-Tiny build-requires (and should run-require)
perl(Type::Tie), perl-Type-Tie build-requires perl(Types::Standard) for
optional tests. Because perl-Type-Tiny uses Type::Tie at run-time
(lib/Type/Tiny.pm:1146), I recommend to put a condition into perl-Type-Tie
package:

--- a/perl-Type-Tie.spec
+++ b/perl-Type-Tie.spec
@@ -25,7 +25,10 @@ BuildRequires:  perl(strict)
 BuildRequires:  perl(warnings)

 # Tests:
+%if !%{defined perl_bootstrap}
+# Build-cycle: perl-Type-Tiny → perl-Type-Tie
 BuildRequires:  perl(Types::Standard)
+%endif
 BuildRequires:  perl(Moose::Util::TypeConstraints)
 BuildRequires:  perl(MooseX::Types::Moose)
 BuildRequires:  perl(Test::Fatal)

I also recommend to add explicit run-time dependency on perl(Type::Tie) to
perl-Type-Tiny package as it is not autogenerated.

[...]

--- Additional comment from Ralf Corsepius on 2017-03-20 11:31:31 GMT ---

Thanks for the patches, applied.

----

The perl-Type-Tiny was patched correctly. Thanks. But it's still necessary to
patch perl-Type-Tie. The necessary change for perl-Type-Tie package is
mentioned above.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
perl-devel mailing list -- perl-devel@lists.fedoraproject.org
To unsubscribe send an email to perl-devel-le...@lists.fedoraproject.org

Reply via email to