Skip Menu |

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

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

History Show all quoted text
  Wed Jun 12 16:08:02 2019 SREZIC [...] cpan.org - Ticket created
Subject: Can't locate Future/IO.pm (0.73)
t/70future-io.t fails if Future::IO is not installed: ... Can't locate Future/IO.pm in @INC (you may need to install the Future::IO module) (@INC contains: . ... ) at t/70future-io.t line 16. BEGIN failed--compilation aborted at t/70future-io.t line 16. t/70future-io.t .............. Dubious, test returned 2 (wstat 512, 0x200) No subtests run ...
  Fri Jun 14 10:55:45 2019 davido [...] cpan.org - Correspondence added
On Wed Jun 12 13:08:02 2019, SREZIC wrote: Show quoted text
> t/70future-io.t fails if Future::IO is not installed: > > ... > Can't locate Future/IO.pm in @INC (you may need to install the > Future::IO module) (@INC contains: . ... ) at t/70future-io.t line 16. > BEGIN failed--compilation aborted at t/70future-io.t line 16. > t/70future-io.t .............. > Dubious, test returned 2 (wstat 512, 0x200) > No subtests run > ...
I have the same behavior when installing IO::Async version 0.73 upgrading from 0.72. Manually installing Future::IO version 0.05 resolves the issue, and allows IO::Async to pass its test suite and install cleanly. I've verified this in Perl 5.16, 5.28, and 5.30.
  Fri Jun 14 10:55:46 2019 The RT System itself - Status changed from 'new' to 'open'
  Fri Jun 14 11:11:45 2019 leonerd-cpan [...] leonerd.org.uk - Correspondence added
Yeah; the test should probably just be conditional on the module existing: plan skip_all => "Future::IO is not available" unless eval { require Future::IO }; -- Paul Evans
  Sun Jun 23 14:05:36 2019 leonerd-cpan [...] leonerd.org.uk - Correspondence added
Patch attached -- Paul Evans
Subject: rt129807.patch
=== modified file 't/70future-io.t' --- t/70future-io.t 2019-04-10 23:12:39 +0000 +++ t/70future-io.t 2019-06-23 18:04:55 +0000 @@ -13,8 +13,9 @@ use IO::Async::Loop; use IO::Async::OS; -use Future::IO; -use Future::IO::Impl::IOAsync; +eval { require Future::IO; } or + plan skip_all => "Future::IO is not available"; +require Future::IO::Impl::IOAsync; use constant AUT => $ENV{TEST_QUICK_TIMERS} ? 0.1 : 1;
  Sun Jun 23 14:05:37 2019 leonerd-cpan [...] leonerd.org.uk - Status changed from 'open' to 'patched'
  Sat Nov 23 12:44:25 2019 leonerd-cpan [...] leonerd.org.uk - Correspondence added
This was released in 0.74 -- Paul Evans
  Sat Nov 23 12:44:25 2019 leonerd-cpan [...] leonerd.org.uk - Status changed from 'patched' to 'resolved'
  Sat Nov 23 12:44:26 2019 leonerd-cpan [...] leonerd.org.uk - Fixed in 0.74 added