search

stevan / promises-perl

An implementation of Promises in Perl
31 stars 29 forks source link

The test suite requires EV.pm to be present, added it to dist.ini's #34

Closed mschilli closed 9 years ago

mschilli commented 10 years ago

TestRequires section. Currently, when I'm installing Promises via cpanm, I get

Failed test 'use Promises::Deferred::EV;'

at t/052-exceptions-ev-anyevent.t line 12.

Tried to use 'Promises::Deferred::EV'.

Error: Can't locate EV.pm in @INC (you may need to install the EV module) (@INC contains: t/lib /home/mschilli/.cpanm/work/1413273423.4595/Promises-0.93/blib/lib /home/mschilli/.cpanm/work/1413273423.4595/Promises-0.93/blib/arch /h

Thanks for this great module!

stevan commented 10 years ago

Actually, ideally we add a guard clause in the test file for EV instead, I will try and get around to that soon.

mschilli commented 10 years ago

Friendly reminder that this still needs fixing.

mschilli commented 9 years ago

Harumph.

stevan commented 9 years ago

Sorry this took so long, it should be fixed on CPAN now (0.94)

mschilli commented 9 years ago

Works like a charm now, thanks for fixing it!