Bug #67193 for Module-Install-TemplateInstallPath: Module::Install dependency missing?

Sun Apr 03 10:11:48 2011 SREZIC [...] cpan.org - Ticket created

Subject:

Module::Install dependency missing?

It seems that the distribution should declare Module::Install as a dependency, otherwise there are a lot of unnecessary fail reports, see http://matrix.cpantesters.org/?dist=Module-Install-TemplateInstallPath&maxver= Regards, Slaven

Mon Apr 04 09:54:26 2011 djerius [...] cpan.org - Correspondence added

On Sun Apr 03 10:11:48 2011, SREZIC wrote: Show quoted text

> It seems that the distribution should declare Module::Install as a > dependency, otherwise there are a lot of unnecessary fail reports, see > http://matrix.cpantesters.org/?dist=Module-Install- > TemplateInstallPath&maxver=

Thanks for noticing the failures. I'm not sure if adding Module::Install is the solution (technically it's supposed to be self contained) as I have other modules which use Module::Install and don't declare it as a dependency (e.g. http://search.cpan.org/~djerius/Params-Validate-Aggregated-0.05/ ) and they don't seem to have the same failures. I'll set up a clean Perl environment and see if I can figure out what's happening. Thanks again.

Mon Apr 04 09:54:27 2011 The RT System itself - Status changed from 'new' to 'open'

Mon Apr 04 19:09:25 2011 slaven [...] rezic.de - Correspondence added

CC:	SREZIC [...] cpan.org
Subject:	Re: [rt.cpan.org #67193] Module::Install dependency missing?
Date:	Tue, 05 Apr 2011 01:08:27 +0200
To:	bug-Module-Install-TemplateInstallPath [...] rt.cpan.org
From:	Slaven Rezic <slaven [...] rezic.de>

"Diab Jerius via RT" <bug-Module-Install-TemplateInstallPath@rt.cpan.org> writes: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=67193 > > > On Sun Apr 03 10:11:48 2011, SREZIC wrote:

>> It seems that the distribution should declare Module::Install as a >> dependency, otherwise there are a lot of unnecessary fail reports, see >> http://matrix.cpantesters.org/?dist=Module-Install- >> TemplateInstallPath&maxver=

> > Thanks for noticing the failures. > > I'm not sure if adding Module::Install is the solution (technically it's > supposed to be self contained) as I have other modules which use > Module::Install and don't declare it as a dependency (e.g. > http://search.cpan.org/~djerius/Params-Validate-Aggregated-0.05/ ) and > they don't seem to have the same failures. I'll set up a clean Perl > environment and see if I can figure out what's happening.

You're right --- as you bundle Module/Install.pm you shouldn't setup a dependency. It seems that the template.t test just needs another directory in @INC Can't locate inc/Module/Install.pm in @INC (@INC contains: /var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA/inc /var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA/blib/lib /var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA/blib/arch /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/site_perl/5.14.0/i386-freebsd /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/site_perl/5.14.0 /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/5.14.0/i386-freebsd /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/5.14.0 .) at Makefile.PL line 1. So apparently "/var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA" (without the "inc") needs to be added to @INC before running "perl Makefile.Pl" in the test. Regards, Slaven -- Slaven Rezic - slaven <at> rezic <dot> de tktimex - time recording tool http://sourceforge.net/projects/ptktools/

Tue Apr 05 10:35:17 2011 dj [...] head.cfa.harvard.edu - Correspondence added

Subject:	Re: [rt.cpan.org #67193] Module::Install dependency missing?
Date:	Tue, 05 Apr 2011 10:35:00 -0400
To:	bug-Module-Install-TemplateInstallPath [...] rt.cpan.org
From:	Diab Jerius <dj [...] head.cfa.harvard.edu>

On Mon, 2011-04-04 at 19:09 -0400, slaven@rezic.de via RT wrote: Show quoted text

> Queue: Module-Install-TemplateInstallPath > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=67193 > > > "Diab Jerius via RT" > <bug-Module-Install-TemplateInstallPath@rt.cpan.org> writes: >

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=67193 > > > > > On Sun Apr 03 10:11:48 2011, SREZIC wrote:

> >> It seems that the distribution should declare Module::Install as a > >> dependency, otherwise there are a lot of unnecessary fail reports, see > >> http://matrix.cpantesters.org/?dist=Module-Install- > >> TemplateInstallPath&maxver=

> > > > Thanks for noticing the failures. > > > > I'm not sure if adding Module::Install is the solution (technically it's > > supposed to be self contained) as I have other modules which use > > Module::Install and don't declare it as a dependency (e.g. > > http://search.cpan.org/~djerius/Params-Validate-Aggregated-0.05/ ) and > > they don't seem to have the same failures. I'll set up a clean Perl > > environment and see if I can figure out what's happening.

> > You're right --- as you bundle Module/Install.pm you shouldn't setup a > dependency. It seems that the template.t test just needs another > directory in @INC > > Can't locate inc/Module/Install.pm in @INC (@INC contains: > /var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA/inc > /var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA/blib/lib > /var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA/blib/arch > /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/site_perl/5.14.0/i386-freebsd > /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/site_perl/5.14.0 > /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/5.14.0/i386-freebsd > /home/cpansand/var/ctps/5140RC0/install/perl-v5.13.11-368-gc2c97af/lib/5.14.0 > .) at Makefile.PL line 1. > > So apparently > "/var/tmp/CPAN-build/Module-Install-TemplateInstallPath-0.01-lmeCaA" > (without the "inc") needs to be added to @INC before running "perl > Makefile.Pl" in the test.

I went through some gyrations trying to get the test working; not enough, apparently! I'll have to set up a pristine Perl distribution and use that for testing. Thanks, Diab

Fri Apr 08 10:36:52 2011 djerius [...] cpan.org - Correspondence added

Module::Install is clever and hard to fool. I ended up creating a "complete" distribution for the tests to keep it happy. New version uploaded; tests seem to be going well.

Fri Apr 08 10:36:52 2011 djerius [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Apr 08 10:37:28 2011 djerius [...] cpan.org - Fixed in 0.02 added