Re: [perl-XML-SAX-Writer] Always use included Module::Install to avoid circular build deps (#906007)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/02/13 16:38, Petr Pisar wrote:
On Tue, Feb 05, 2013 at 04:19:56PM +0000, Paul Howarth wrote:
commit b220aa08f8f4d564f70d052ec9adb2d06c48bc
Author: Paul Howarth <paul@xxxxxxxxxxxx>
Date:   Tue Feb 5 16:19:19 2013 +0000

     Always use included Module::Install to avoid circular build deps (#906007)

  perl-XML-SAX-Writer.spec |   20 +++++++-------------
  1 files changed, 7 insertions(+), 13 deletions(-)
Can you show me the cycle? I cannot find why Module::Install::* stuff would
need XML::SAX::Writer.

There's a lot of them. Remember, these are build dependency cycles, not regular dependency cycles, so things pulled in by test suites take effect. Some samples:

perl-XML-Twig->perl-XML-SAX-Writer->perl-Module-Install-Repository->perl-Path-Class->perl-Test-Perl-Critic->perl-Perl-Critic->perl-List-MoreUtils->perl-Test-LeakTrace->perl-Test-Valgrind->perl-XML-Twig

perl-prefork->perl-Perl-MinimumVersion->perl-Perl-Critic->perl-List-MoreUtils->perl-Test-LeakTrace->perl-Test-Valgrind->perl-XML-Twig->perl-XML-SAX-Writer->perl-Module-Install->perl-Module-ScanDeps->perl-prefork

perl-XML-Twig->perl-XML-SAX-Writer->perl-Module-Install->perl-Test-MinimumVersion->perl-Perl-MinimumVersion->perl-Perl-Critic->perl-List-MoreUtils->perl-Test-LeakTrace->perl-Test-Valgrind->perl-XML-Twig

There's probably more than one place to have broken these cycles, but this way fits in with upstream's build process and doesn't require any bootstrapping conditionals or extra builds at the end of bootstrapping.

Paul.
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/perl-devel



[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Legacy Announce]     [Fedora PHP Devel]     [Kernel Devel]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Big List of Linux Books]     [Gimp]     [Yosemite Information]
  Powered by Linux