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@city-fan.org 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.