Skip Menu |

This queue is for tickets about the Module-Release CPAN distribution.

Report information
The Basics
Id: 37159
Status: rejected
Priority: 0/
Queue: Module-Release
People
Owner: Nobody in particular
Requestors: jbisbee [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Jun 27 04:18:50 2008 jbisbee [...] cpan.org - Ticket created
Subject: Makefile.PL Issues
There are two issues with Module::Release's Makefile.PL 1. 'PM' is Incorrect -------------------- Currently 'PM' => { 'lib/Release.pm' => '$(INST_LIBDIR)/Release.pm', }, And needs to change to 'PM' => { 'lib/Release.pm' => '$(INST_LIBDIR)/Module/Release.pm', }, This doesn't affect the actual install of the module, but its does effect the manpage thats created. jbisbee@bizb:~/Module-Release-1.20$ perl Makefile.PL PREFIX=~/tmp jbisbee@bizb:~/.cpan/build/Module-Release-1.20-AJgzB4$ make install Installing /Users/jbisbee/tmp/lib/perl5/site_perl/Module/Release.pm Installing /Users/jbisbee/tmp/share/man/man1/release.1 -> Installing /Users/jbisbee/tmp/share/man/man3/Release.3pm Installing /Users/jbisbee/tmp/bin/release It should be Module::Relase.3pm not not just Release.3pm. man Module::Release fails with the current Makefile.PL 2. Module::CPANTS::Analyze needs to be added to PREREQ_PM cpants_lint.pl is now called during a 'release' so it makes sense that Module::CPANTS::Analyze should be required. 'PREREQ_PM' => { 'ConfigReader::Simple' => 0, 'Net::FTP' => 0, 'HTTP::Message' => 0, 'IO::Null' => 0, 'Crypt::SSLeay' => 0, 'File::Temp' => 0, -> 'Module::CPANTS::Anaylze' => 0, }, Great idea to include CPANTS btw :) -- Jeff Bisbee / jbisbee@cpan.org
  Fri Jun 27 04:24:30 2008 jbisbee [...] cpan.org - Correspondence added
On Fri Jun 27 04:18:50 2008, JBISBEE wrote: Show quoted text
> 2. Module::CPANTS::Analyze needs to be added to PREREQ_PM > > cpants_lint.pl is now called during a 'release' so it makes sense that > Module::CPANTS::Analyze should be required. > > 'PREREQ_PM' => { > 'ConfigReader::Simple' => 0, > 'Net::FTP' => 0, > 'HTTP::Message' => 0, > 'IO::Null' => 0, > 'Crypt::SSLeay' => 0, > 'File::Temp' => 0, > -> 'Module::CPANTS::Anaylze' => 0, > },
Ops, should be 'Anaylse' and not 'Analyze' (or Anaylze as I misspelled it) -- Jeff Bisbee / jbisbee@cpan.org
  Fri Jun 27 04:24:31 2008 jbisbee [...] cpan.org - Status changed from 'new' to 'open'
  Fri Jun 27 13:49:53 2008 bdfoy [...] cpan.org - Correspondence added
1. The PM section to WriteMakefile is just fine. I needed to add a MAN3POD section though. 2. The release script uses cpants_lint.pl, and external program. I need to make that a conditional use somehow, but it doesn't mean that I need to make CPANTS a dependency. Thanks though :)
  Fri Jun 27 13:49:57 2008 bdfoy [...] cpan.org - Status changed from 'open' to 'rejected'