Beefy Boxes and Bandwidth Generously Provided by pair Networks
Do you know where your variables are?

Re^2: Cannot read socket fh in Net::Daemon test

by mnooning (Beadle)
on Feb 17, 2011 at 20:27 UTC ( #888789=note: print w/replies, xml ) Need Help??

in reply to Re: Cannot read socket fh in Net::Daemon test
in thread Cannot read socket fh in Net::Daemon test

Some answers to this are on the bug ticket, and have come from a combination of Daniel Macks and mcmahon. For one thing, threads on Windows have been supplanted by ithreads after Perl 5.10. In addition, a nice patch by Gaspar Chilingarov seems to have made a lot of headway today towards solving the problem.

The t/thread.t tests are simply skipped if version >= 5.10. The only remaining problem is in telling the test harness/TAP - that it is okay not to have any tests for this. Just printing out "ok" to STDOUT or STDERR before exiting does not work. I will break out my old book - Perl Testing .. A Developer's Notebook - and figure this out.

I should be able to get it from here, but if anyone could spare me the reading, that would be great.

  • Comment on Re^2: Cannot read socket fh in Net::Daemon test

Replies are listed 'Best First'.
Re^3: Cannot read socket fh in Net::Daemon test
by Anonymous Monk on Feb 17, 2011 at 23:53 UTC

    Got it. Just for completeness, I will list the last problem and its solution.

    I was getting the error below when tests were being skipped completely.

    Parse errors: No plan found in TAP output

    The solution was to use Test::More in t/thread.t, and add " plan(skip_all => $message); at the appropriate place, which was just before exiting the file, once it became known that all tests were to be skipped.


Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://888789]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others surveying the Monastery: (8)
As of 2018-01-17 22:17 GMT
Find Nodes?
    Voting Booth?
    How did you see in the new year?

    Results (206 votes). Check out past polls.