Skip Menu |

This queue is for tickets about the Test-Harness CPAN distribution.

Report information
The Basics
Id: 105524
Status: rejected
Priority: 0/
Queue: Test-Harness
People
Owner: Nobody in particular
Requestors: develop [...] traveljury.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 3.35
Fixed in: (no value)

History Show all quoted text
  Sat Jun 27 07:37:18 2015 develop [...] traveljury.com - Ticket created
Subject: TAP::Harness::JUnit not loading in perl 5.8.9
This command works on perl 5.10 and above: prove --harness=TAP::Harness::JUnit -l t/*/*.t On perl 5.8.9 it fails with: Can't locate object method "new" via package "TAP::Harness::JUnit" at /Users/clinton/perl5/perlbrew/perls/perl-5.8.9/lib/site_perl/5.8.9/TAP/Harness/JUnit.pm line 145. Loading TAP::Harness::Junit first works: perl -MTAP::Harness::JUnit /Users/clinton/perl5/perlbrew/perls/perl-5.8.9/bin/prove --harness=TAP::Harness::JUnit -l t/*/*.t but preloading the file with prove -M fails with the same "can't find new" message: prove -MTAP::Harness::JUnit --harness=TAP::Harness::JUnit -l t/*/*.t Not sure if this is a bug in TAP::Harness or in TAP::Harness::JUnit
  Fri Jul 03 12:26:13 2015 develop [...] traveljury.com - Correspondence added
OK I've tracked this down to a problem with base.pm This line in T::H::JUnit has "use base 'TAP::Harness" https://metacpan.org/source/JLAVALLEE/TAP-Harness-JUnit-0.42/lib/TAP/Harness/JUnit.pm#L32 I upgraded "base" to v2.18, and the problem seems to be resolved. So I suppose TAP::Harness just needs to bump the minimum version of base to 2.18
  Fri Jul 03 20:55:56 2015 LEONT [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #105524] TAP::Harness::JUnit not loading in perl 5.8.9
Date: Sat, 4 Jul 2015 02:55:22 +0200
To: bug-Test-Harness [...] rt.cpan.org
From: Leon Timmermans <leont [...] cpan.org>
On Fri, Jul 3, 2015 at 6:26 PM, Clinton Gormley via RT < bug-Test-Harness@rt.cpan.org> wrote: Show quoted text
> OK I've tracked this down to a problem with base.pm > > This line in T::H::JUnit has "use base 'TAP::Harness" > https://metacpan.org/source/JLAVALLEE/TAP-Harness-JUnit-0.42/lib/TAP/Harness/JUnit.pm#L32 > > I upgraded "base" to v2.18, and the problem seems to be resolved. > > So I suppose TAP::Harness just needs to bump the minimum version of base > to 2.18 >
Test::Harness is part of the topmost set of dependencies on CPAN. I'm not even sure I can increase that dependency without repercussions (for circular dependency reasons), I'm certainly not going to add it without understanding why it's necessary. But regardless of that, wouldn't adding such a dependency to TAP::Harness::JUnit be a much more obvious solution? Leon
  Fri Jul 03 20:55:57 2015 The RT System itself - Status changed from 'new' to 'open'
  Sat Jul 04 10:51:47 2015 develop [...] traveljury.com - Correspondence added
Show quoted text
> But regardless of that, wouldn't adding such a dependency to > TAP::Harness::JUnit be a much more obvious solution?
You are, of course, right :) I didn't check whether Test::Harness relied on base.pm already. I'll open an issue there
  Sat Jul 04 10:51:57 2015 develop [...] traveljury.com - Status changed from 'open' to 'rejected'
  Sat Jul 04 15:54:56 2015 ether [...] cpan.org - Reference by ticket #105657 added