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

[Paul Howarth]

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 at 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.