Bug #84794 for App-cpanminus-reporter: Missing dependency on Test::Reporter::Transport::Metabase?

Mon Apr 22 18:38:45 2013 nomad [...] null.net - Ticket created

Subject:	Missing dependency on Test::Reporter::Transport::Metabase?
Date:	Tue, 23 Apr 2013 10:38:11 +1200
To:	bug-App-cpanminus-reporter [...] rt.cpan.org
From:	Mark Lawrence <nomad [...] null.net>

I installed cpanm-reporter 0.06: mark@lifebook:~/src$ cpanm App::cpanminus::reporter --> Working on App::cpanminus::reporter Fetching http://www.cpan.org/authors/id/G/GA/GARU/App-cpanminus-reporter-0.06.tar.gz ... OK Then ran it after installing something else and observed the following: mark@lifebook:~/src$ cpanm XXX --> Working on XXX Fetching http://www.cpan.org/authors/id/I/IN/INGY/XXX-0.18.tar.gz ... OK Configuring XXX-0.18 ... OK Building and testing XXX-0.18 ... OK Successfully installed XXX-0.18 1 distribution installed mark@lifebook:~/src$ cpanm-reporter Parsing /home/mark/.cpanm/latest-build/build.log... sending: (http://www.cpan.org/authors/id/I/IN/INGY/XXX-0.18.tar.gz, INGY, XXX-0.18, PASS) Test::Reporter: could not load 'Test::Reporter::Transport::Metabase' Can't locate Test/Reporter/Transport/Metabase.pm in @INC (@INC contains: /opt/perlbrew/perls/perl-5.14.2/lib/site_perl/5.14.2/x86_64-linux /opt/perlbrew/perls/perl-5.14.2/lib/site_perl/5.14.2 /opt/perlbrew/perls/perl-5.14.2/lib/5.14.2/x86_64-linux /opt/perlbrew/perls/perl-5.14.2/lib/5.14.2 .) at (eval 27) line 1. at /opt/perlbrew/perls/perl-5.14.2/lib/site_perl/5.14.2/App/cpanminus/reporter.pm line 271. After installing Test::Reporter::Transport::Metabase, cpanm-reporter worked as expected: mark@lifebook:~/src$ cpanm-reporter Parsing /home/mark/.cpanm/latest-build/build.log... sending: (http://www.cpan.org/authors/id/D/DA/DAGOLDEN/Metabase-Client-Simple-0.009.tar.gz, DAGOLDEN, Metabase-Client-Simple-0.009, PASS) sending: (http://www.cpan.org/authors/id/D/DA/DAGOLDEN/CPAN-Testers-Report-1.999001.tar.gz, DAGOLDEN, CPAN-Testers-Report-1.999001, PASS) sending: (http://www.cpan.org/authors/id/D/DA/DAGOLDEN/Test-Reporter-Transport-Metabase-1.999008.tar.gz, DAGOLDEN, Test-Reporter-Transport-Metabase-1.999008, PASS) Finished. -- Mark Lawrence

Wed May 08 01:59:33 2013 garu [...] cpan.org - Correspondence added

Hi Mark, thanks for the report! The issue here is that, even though Metabase is *the* most used transport system (and the only one currently accepted by the CPAN Testers servers), there are cases where you go with different transports, such as files or sockets, in order to transfer them to a temporary location and push them asynchronously to the CPAN Testers server (via Metabase) later on, and likely through another script. So yes, it's a pretty safe bet that the vast majority of 'cpanm-reporter' users will need to have the Metabase transport installed. But no, it's not really a dependency for cpanm-reporter itself. That said, I added 'Test::Reporter::Transport::Metabase' to the "recommends" section of the Makefile.PL. Hopefully that will work out for both use cases (though I know Miyagawa is still adjusting the defaults on recommended dists in cpanm). Let's see how it goes. If it's still not enough, I'll force Test::Reporter::Transport::Metabase as a hard dep, as it is better to make things easier for the common use case than it is for people that are working extra magic :) Thanks again for the report! On Mon Apr 22 18:38:45 2013, nomad@null.net wrote: Show quoted text

> I installed cpanm-reporter 0.06: > > mark@lifebook:~/src$ cpanm App::cpanminus::reporter > --> Working on App::cpanminus::reporter > Fetching http://www.cpan.org/authors/id/G/GA/GARU/App-cpanminus- > reporter-0.06.tar.gz ... OK > > Then ran it after installing something else and observed the > following: > > mark@lifebook:~/src$ cpanm XXX > --> Working on XXX > Fetching http://www.cpan.org/authors/id/I/IN/INGY/XXX-0.18.tar.gz > ... OK > Configuring XXX-0.18 ... OK > Building and testing XXX-0.18 ... OK > Successfully installed XXX-0.18 > 1 distribution installed > > mark@lifebook:~/src$ cpanm-reporter > Parsing /home/mark/.cpanm/latest-build/build.log... > sending: (http://www.cpan.org/authors/id/I/IN/INGY/XXX- > 0.18.tar.gz, INGY, XXX-0.18, PASS) > Test::Reporter: could not load > 'Test::Reporter::Transport::Metabase' > Can't locate Test/Reporter/Transport/Metabase.pm in @INC (@INC > contains: /opt/perlbrew/perls/perl- > 5.14.2/lib/site_perl/5.14.2/x86_64-linux /opt/perlbrew/perls/perl- > 5.14.2/lib/site_perl/5.14.2 /opt/perlbrew/perls/perl- > 5.14.2/lib/5.14.2/x86_64-linux /opt/perlbrew/perls/perl- > 5.14.2/lib/5.14.2 .) at (eval 27) line 1. > > at /opt/perlbrew/perls/perl- > 5.14.2/lib/site_perl/5.14.2/App/cpanminus/reporter.pm line 271. > > After installing Test::Reporter::Transport::Metabase, cpanm-reporter > worked as expected: > > mark@lifebook:~/src$ cpanm-reporter > Parsing /home/mark/.cpanm/latest-build/build.log... > sending: (http://www.cpan.org/authors/id/D/DA/DAGOLDEN/Metabase- > Client-Simple-0.009.tar.gz, DAGOLDEN, Metabase-Client-Simple-0.009, > PASS) > sending: (http://www.cpan.org/authors/id/D/DA/DAGOLDEN/CPAN- > Testers-Report-1.999001.tar.gz, DAGOLDEN, CPAN-Testers-Report- > 1.999001, PASS) > sending: (http://www.cpan.org/authors/id/D/DA/DAGOLDEN/Test- > Reporter-Transport-Metabase-1.999008.tar.gz, DAGOLDEN, Test- > Reporter-Transport-Metabase-1.999008, PASS) > Finished. >

Wed May 08 01:59:33 2013 The RT System itself - Status changed from 'new' to 'open'

Wed May 08 01:59:33 2013 garu [...] cpan.org - Status changed from 'open' to 'resolved'