Skip Menu |

This queue is for tickets about the IO-Async-SSL CPAN distribution.

Report information
The Basics
Id: 128418
Status: patched
Priority: 0/
Queue: IO-Async-SSL
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.22
Fixed in: (no value)

History Show all quoted text
  Mon Feb 04 15:55:39 2019 SREZIC [...] cpan.org - Ticket created
Subject: Undeclared dependency Test::Identity
t/01upgrade.t and t/05utf8.t fail if Test::Identity is not installed: ... Can't locate Test/Identity.pm in @INC (you may need to install the Test::Identity module) (@INC contains: ... .) at t/01upgrade.t line 7. BEGIN failed--compilation aborted at t/01upgrade.t line 7. t/01upgrade.t ........... Dubious, test returned 2 (wstat 512, 0x200) No subtests run ... Can't locate Test/Identity.pm in @INC (you may need to install the Test::Identity module) (@INC contains: ... .) at t/05utf8.t line 8. BEGIN failed--compilation aborted at t/05utf8.t line 8. t/05utf8.t .............. Dubious, test returned 2 (wstat 512, 0x200) No subtests run ...
  Tue Feb 05 07:54:30 2019 leonerd-cpan [...] leonerd.org.uk - Correspondence added
On Mon Feb 04 15:55:39 2019, SREZIC wrote: Show quoted text
> t/01upgrade.t and t/05utf8.t fail if Test::Identity is not installed:
Oops, indeed so. I guess rarely that comes up in practice because the IO-Async dist itself depends on Test::Identity, so it's likely that would already be available. But yes, I've added it to Build.PL now. Thanks, -- Paul Evans
  Tue Feb 05 07:54:30 2019 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 05 07:54:31 2019 leonerd-cpan [...] leonerd.org.uk - Status changed from 'open' to 'patched'
  Tue Feb 05 17:26:03 2019 SREZIC [...] cpan.org - Correspondence added
On 2019-02-05 07:54:30, PEVANS wrote: Show quoted text
> On Mon Feb 04 15:55:39 2019, SREZIC wrote:
> > t/01upgrade.t and t/05utf8.t fail if Test::Identity is not installed:
> > Oops, indeed so. I guess rarely that comes up in practice because the > IO-Async dist itself depends on Test::Identity, so it's likely that > would already be available.
But it's only a build requirement. So if a user has configured something like build_requires_install_policy=no in his/her cpan client, then Test::Identity won't be installed together with IO-Async.