Bug #30963 for Mail-SpamCannibal: Use configure_requires for IPTables::IPv4::DBTarpit::Inst dependency

Sun Nov 25 06:18:35 2007 SREZIC [...] cpan.org - Ticket created

Subject:

Use configure_requires for IPTables::IPv4::DBTarpit::Inst dependency

See http://module-build.sourceforge.net/META-spec-blead.html#configure_requires for more information about configure_requires. At least CPAN.pm uses the value when installing distributions. Regards, Slaven

Mon Dec 10 16:04:26 2007 michael [...] bizsystems.com - Correspondence added

EXTRA_META tag added in Makefile.PL v0.84 released.

Mon Dec 10 16:04:34 2007 The RT System itself - Status changed from 'new' to 'open'

Mon Dec 10 16:04:40 2007 michael [...] bizsystems.com - Status changed from 'open' to 'resolved'

Mon Dec 10 17:20:59 2007 SREZIC [...] cpan.org - Correspondence added

From:

SREZIC [...] cpan.org

On Mon Dec 10 16:04:26 2007, MIKER wrote: Show quoted text

> EXTRA_META tag added in Makefile.PL > > v0.84 released.

Unfortunately the indentation in the value part is missing. See the attached patch for a fix. Regards, Slaven

#### Patch data follows #### diff -up '../build/Mail-SpamCannibal-0.84-yhJ7RL/Makefile.PL' 'Mail-SpamCannibal-0.84/Makefile.PL' Index: ./Makefile.PL --- ./Makefile.PL Mon Dec 10 21:59:31 2007 +++ ./Makefile.PL Mon Dec 10 23:17:53 2007 @@ -90,7 +90,7 @@ foreach(keys %xmodules) { my %makeparms = ( 'EXTRA_META' => q|configure_requires: -IPTables::IPv4::DBTarpit::Inst: 0 + IPTables::IPv4::DBTarpit::Inst: 0 |, 'NAME' => 'Mail::SpamCannibal', 'VERSION_FROM' => 'SpamCannibal.pm', # finds $VERSION #### End of Patch data ####

Mon Dec 10 17:21:04 2007 The RT System itself - Status changed from 'resolved' to 'open'

Mon Dec 10 18:58:34 2007 michael [...] insulin-pumpers.org - Correspondence added

Subject:	Re: [rt.cpan.org #30963] Use configure_requires for IPTables::IPv4::DBTarpit::Inst dependency
Date:	Mon, 10 Dec 2007 15:58:20 -0800
To:	bug-Mail-SpamCannibal [...] rt.cpan.org
From:	michael [...] insulin-pumpers.org

Show quoted text

> > Queue: Mail-SpamCannibal > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=30963 > > > On Mon Dec 10 16:04:26 2007, MIKER wrote:

> > EXTRA_META tag added in Makefile.PL > > > > v0.84 released.

> > Unfortunately the indentation in the value part is missing. See the > attached patch for a fix. > > Regards, > Slaven > >

Ok, thanks. Perhaps you can help with the other bit about the indexer and the sandbox that perl runs in when looking at new distro's. It seems that this line is not being expanded in the sandbox $VERSION = do 'something.pm'; can I use 'require' and get it to work, or perhaps something like... my $var = require 'something.pm'; $VERSION = $var; ?? I need to extract the version variable from another pm that is used globally and returns the $VERSION number. Michael

Tue Dec 11 16:58:09 2007 slaven [...] rezic.de - Correspondence added

CC:	SREZIC [...] cpan.org
Subject:	Re: [rt.cpan.org #30963] Use configure_requires for IPTables::IPv4::DBTarpit::Inst dependency
Date:	11 Dec 2007 22:52:07 +0100
To:	bug-Mail-SpamCannibal [...] rt.cpan.org
From:	Slaven Rezic <slaven [...] rezic.de>

"michael@insulin-pumpers.org via RT" <bug-Mail-SpamCannibal@rt.cpan.org> writes: Show quoted text

> <URL: http://rt.cpan.org/Ticket/Display.html?id=30963 > >

> > > > Queue: Mail-SpamCannibal > > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=30963 > > > > > On Mon Dec 10 16:04:26 2007, MIKER wrote:

> > > EXTRA_META tag added in Makefile.PL > > > > > > v0.84 released.

> > > > Unfortunately the indentation in the value part is missing. See the > > attached patch for a fix. > > > > Regards, > > Slaven > > > >

> > Ok, thanks. > > Perhaps you can help with the other bit about the indexer and the > sandbox that perl runs in when looking at new distro's. > > It seems that this line is not being expanded in the sandbox > > $VERSION = do 'something.pm'; > > can I use 'require' and get it to work, or perhaps something like... > > my $var = require 'something.pm'; > > $VERSION = $var; > > ?? > > I need to extract the version variable from another pm that is used > globally and returns the $VERSION number. >

This should usually work: require 'something.pm'; $Something::VERSION; # if something.pm defines the package Something Regards, Slaven -- Slaven Rezic - slaven <at> rezic <dot> de babybike - routeplanner for cyclists in Berlin handheld (e.g. Compaq iPAQ with Linux) version of bbbike http://bbbike.sourceforge.net

Wed Sep 03 19:40:04 2008 michael [...] bizsystems.com - Correspondence added

I believe this bug was resolved 12-07 so I'm setting to "resolved"

Wed Sep 03 19:40:08 2008 michael [...] bizsystems.com - Status changed from 'open' to 'resolved'