Skip Menu |
 

This queue is for tickets about the Net-Radio-Modem CPAN distribution.

Report information
The Basics
Id: 112889
Status: open
Priority: 0/
Queue: Net-Radio-Modem

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.002
Fixed in: (no value)



Subject: Pod test fails
Download (untitled) / with headers
text/plain 379b
Test output: # Failed test 'POD test for blib/lib/Net/Radio/Modem.pm' # at /usr/perl5.22.1p/lib/site_perl/5.22.1/Test/Pod.pm line 187. # blib/lib/Net/Radio/Modem.pm (99): alternative text 'Net::Radio::Modem::Adapter/Alias' contains non-escaped | or / # Looks like you failed 1 test of 4. t/pod.t ........... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/4 subtests
Download (untitled) / with headers
text/plain 988b
On 2016-03-09 22:37:38, SREZIC wrote: Show quoted text
> Test output: > > # Failed test 'POD test for blib/lib/Net/Radio/Modem.pm' > # at /usr/perl5.22.1p/lib/site_perl/5.22.1/Test/Pod.pm line 187. > # blib/lib/Net/Radio/Modem.pm (99): alternative text > 'Net::Radio::Modem::Adapter/Alias' contains non-escaped | or / > # Looks like you failed 1 test of 4. > t/pod.t ........... > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/4 subtests
Please don't allow pod, critic or kwalitee tests to run for normal user installs, as they make the installation fail unnecessarily when newer versions of the tester module introduces errors that you cannot anticipate. Also spelling tests are dependent on what dictionaries are installed locally, and critic tests use all plugins that are installed, both of which cannot be predicted in advance. Either move them to xt/, or guard them with: plan skip_all => "These tests are for authors only!" unless $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};
Download (untitled) / with headers
text/plain 1.7k
On Thu Mar 10 12:22:34 2016, ETHER wrote: Show quoted text
> On 2016-03-09 22:37:38, SREZIC wrote:
> > Test output: > > > > # Failed test 'POD test for blib/lib/Net/Radio/Modem.pm' > > # at /usr/perl5.22.1p/lib/site_perl/5.22.1/Test/Pod.pm line 187. > > # blib/lib/Net/Radio/Modem.pm (99): alternative text > > 'Net::Radio::Modem::Adapter/Alias' contains non-escaped | or / > > # Looks like you failed 1 test of 4. > > t/pod.t ........... > > Dubious, test returned 1 (wstat 256, 0x100) > > Failed 1/4 subtests
> > > Please don't allow pod, critic or kwalitee tests to run for normal > user installs, as they make the installation fail unnecessarily when > newer versions of the tester module introduces errors that you cannot > anticipate. Also spelling tests are dependent on what dictionaries are > installed locally, and critic tests use all plugins that are > installed, both of which cannot be predicted in advance. > > Either move them to xt/, or guard them with: > > plan skip_all => "These tests are for authors only!" unless > $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};
https://metacpan.org/requires/distribution/Net-Radio-Modem?sort=[[2,1]] says: * Net-Radio-Modem-Adapter-oFono-0.001 and https://metacpan.org/requires/distribution/Net-Radio-Modem-Adapter-oFono?sort=[[2,1]] says: * Net-Radio-Location-SUPL-Test-0.001 Currently there're no users outside the department of Vodafone where it was developed for. The hardware and software requirements running this piece of ... is incredible high. So - if you're bored - why don't take a minute and study the distribution you're adding your 5 cents? Autogenerated PR's as SREZIC's are more or less sane, but ... seriously? For the records: I won't fix it since there are no users - wasted time.
Download (untitled) / with headers
text/plain 2.8k
On 2016-03-10 12:35:31, REHSACK wrote: Show quoted text
> On Thu Mar 10 12:22:34 2016, ETHER wrote:
> > On 2016-03-09 22:37:38, SREZIC wrote:
> > > Test output: > > > > > > # Failed test 'POD test for blib/lib/Net/Radio/Modem.pm' > > > # at /usr/perl5.22.1p/lib/site_perl/5.22.1/Test/Pod.pm line 187. > > > # blib/lib/Net/Radio/Modem.pm (99): alternative text > > > 'Net::Radio::Modem::Adapter/Alias' contains non-escaped | or / > > > # Looks like you failed 1 test of 4. > > > t/pod.t ........... > > > Dubious, test returned 1 (wstat 256, 0x100) > > > Failed 1/4 subtests
> > > > > > Please don't allow pod, critic or kwalitee tests to run for normal > > user installs, as they make the installation fail unnecessarily when > > newer versions of the tester module introduces errors that you cannot > > anticipate. Also spelling tests are dependent on what dictionaries > > are > > installed locally, and critic tests use all plugins that are > > installed, both of which cannot be predicted in advance. > > > > Either move them to xt/, or guard them with: > > > > plan skip_all => "These tests are for authors only!" unless > > $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};
> > https://metacpan.org/requires/distribution/Net-Radio- > Modem?sort=[[2,1]] says: > > * Net-Radio-Modem-Adapter-oFono-0.001 > > and https://metacpan.org/requires/distribution/Net-Radio-Modem- > Adapter-oFono?sort=[[2,1]] says: > > * Net-Radio-Location-SUPL-Test-0.001 > > Currently there're no users outside the department of Vodafone where > it was developed for. The hardware and software requirements running > this piece of ... is incredible high. > > So - if you're bored - why don't take a minute and study the > distribution you're adding your 5 cents? > > Autogenerated PR's as SREZIC's are more or less sane, but ... > seriously? > > For the records: I won't fix it since there are no users - wasted > time.
The problem with every CPAN distribution with failing tests is that it may appear on the beforemaintrelease page: http://analysis.cpantesters.org/beforemaintrelease?pair=5.22.1:5.23.8 And in fact, I stumbled over your distribution because it was on this page (maybe now it's vanished --- may happen if in the meantime a fail was generated for the older perl). Every entry on this page means additional effort for the people who look over the page and analyze whether it's a regression caused by the newer perl or just a problem of the CPAN distribution. If it's the latter, then I create a RT ticket and link this ticket on the beforemaintrelease page, so it's known that this problem was acted on. So instead of rejecting this ticket it would be far more helpful to remove the distribution from CPAN if there are really no users (and hoping that nobody is smoking backpan distributions), or to make the tiny change (i.e. remove the pod test completely) and re-release the module. Regards, Slaven


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.