Bug #79279 for Net-SNMPTrapd: Net::SNMP missing from (test) prerequisites

Wed Aug 29 04:05:05 2012 d.thomas [...] its.uq.edu.au - Ticket created

Subject:

Net::SNMP missing from (test) prerequisites

Show quoted text

>Net::SNMPTrapd needs the Net::SNMP module to perform >the full set of tests

you could add a 'test_requires' to META.yml and this to META.json "test" : { "requires" : { "Net::SNMP" : "0", } }

Wed Aug 29 08:29:46 2012 vinsworldcom [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #79279] Net::SNMP missing from (test) prerequisites
Date:	Wed, 29 Aug 2012 08:29:36 -0400
To:	bug-Net-SNMPTrapd [...] rt.cpan.org
From:	Vince <vinsworldcom [...] gmail.com>

Will update next week On Aug 29, 2012 4:05 AM, "d.thomas@its.uq.edu.au via RT" < bug-Net-SNMPTrapd@rt.cpan.org> wrote: Show quoted text

> Wed Aug 29 04:05:05 2012: Request 79279 was acted upon. > Transaction: Ticket created by d.thomas@its.uq.edu.au > Queue: Net-SNMPTrapd > Subject: Net::SNMP missing from (test) prerequisites > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: d.thomas@its.uq.edu.au > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=79279 > > >

> >Net::SNMPTrapd needs the Net::SNMP module to perform > >the full set of tests

> you could add a 'test_requires' to META.yml > > and this to META.json > "test" : { > "requires" : { > "Net::SNMP" : "0", > } > } > >

Wed Aug 29 08:29:47 2012 The RT System itself - Status changed from 'new' to 'open'

Mon Sep 03 21:23:51 2012 VINSWORLD [...] cpan.org - Correspondence added

On Wed Aug 29 04:05:05 2012, d.thomas@its.uq.edu.au wrote: Show quoted text

> >Net::SNMPTrapd needs the Net::SNMP module to perform > >the full set of tests

> you could add a 'test_requires' to META.yml > > and this to META.json > "test" : { > "requires" : { > "Net::SNMP" : "0", > } > }

The test.pl file should test if Net::SNMP is installed and fail gracefully if not. Since this isn't a requirement for installation of the module, I don't include it in the Makefile.PL PREREQ_PM list. The META files are generated at 'make dist' time, so I won't edit them manually. Do you have a recommendation as to how to add Net::SNMP as a test-only suggestion (not a requirement) using the EU::MM module my Makefile.PL uses? If it can't be done and the tests fail gracefully without Net::SNMP, I won't bother adding it.

Thu Sep 06 20:33:02 2012 VINSWORLD [...] cpan.org - Correspondence added

On Mon Sep 03 21:23:51 2012, VINSWORLD wrote: Show quoted text

> On Wed Aug 29 04:05:05 2012, d.thomas@its.uq.edu.au wrote:

> > >Net::SNMPTrapd needs the Net::SNMP module to perform > > >the full set of tests

> > you could add a 'test_requires' to META.yml > > > > and this to META.json > > "test" : { > > "requires" : { > > "Net::SNMP" : "0", > > } > > }

> > The test.pl file should test if Net::SNMP is installed and fail > gracefully if not. Since this isn't a requirement for installation of > the module, I don't include it in the Makefile.PL PREREQ_PM list. > > The META files are generated at 'make dist' time, so I won't edit them > manually. > > Do you have a recommendation as to how to add Net::SNMP as a test-only > suggestion (not a requirement) using the EU::MM module my Makefile.PL > uses? If it can't be done and the tests fail gracefully without > Net::SNMP, I won't bother adding it. >

No response - closing this ticket.

Thu Sep 06 20:35:01 2012 VINSWORLD [...] cpan.org - Correspondence added

No response from initiator. The Net::SNMP module is required for advanced tests only and fails gracefully if not found. This will not be included in PREREQ_PM in the EU::MM Makefile.PL and manually editing the JSON file as suggested will not work when creating the distribution.

Thu Sep 06 20:35:01 2012 VINSWORLD [...] cpan.org - Status changed from 'open' to 'rejected'