shogo82148 / Redis-Fast

fast perl binding for Redis database
https://metacpan.org/release/Redis-Fast
Other
25 stars 21 forks source link

Undeclared dependencies (Redis-Fast-0.17) #55

Closed eserte closed 7 years ago

eserte commented 8 years ago

t/02-responses.t fails if Parallel::ForkManager or Test::UNIXSock are not installed. Probably these should be either declared or the test should be skipped if these are missing.

shogo82148 commented 8 years ago

Parallel::ForkManager and Test::UNIXSock is required for test in Redis::Fast 0.17. I declared these modules in cpanfile and META.json.

could you try to resolve dependencies again.

eserte commented 8 years ago

Problem seen on multiple cpantesters systems, not only on mine: http://matrix.cpantesters.org/?dist=Redis-Fast%200.17;reports=1

eserte commented 8 years ago

After running Build.PL manually (without CPAN.pm) I see the following in MYMETA.json — note the absence of all test prereqs:

   "prereqs" : {
      "build" : {
         "requires" : {
            "ExtUtils::CBuilder" : "0"
         }
      },
      "configure" : {
         "requires" : {
            "Module::Build" : "0.4005",
            "Module::Build::XSUtil" : "0.14"
         }
      }
   },
shogo82148 commented 8 years ago

released Redis::Fast 0.18 and this issue is fixed.