| Subject: | TEST_REQUIRES should not add "Thread" for non-threaded perls |
While t/thread.t correctly skips the test script on non-threaded perls, there's another problem which is visible in this sample report:
http://www.cpantesters.org/cpan/report/f5a9ecce-be90-11e5-8596-82020359f3db
The build_requires line lists:
! Thread 0 broken
This means if this was a non-pass report, then the report wouldn't be sent because of broken dependencies.
Probably the TEST_REQUIRES line should be written (untested):
( do { require Config; $Config::Config{usethreads} } ? ("Thread" => 0) : () ),