> > 2. t/08_errors.........FAILED test 21
> > 
> > Test 21 is a POE::Wheel::SocketFactory test with bogus SuccessEvent and
> > FailureEvent values.  It seems to be failing when it attempts to listen
on
> > the socket without first binding it.  t/08_errors.t does not provide
> > SocketFactory with a BindPort argument for test 21, but when I add one,
it
> > succeeds just fine.  I suppose the test should find a free port somehow
and
> > include it, but I don't know what issues this raises on other platforms.
> > The listen() on Solaris appears to have no issue with the same code.

> Test 21 is for UNIX domain sockets.  It seems odd that applying a
> BindPort would fix it.  Does Cygwin support UNIX sockets at all?

> I haven't applied this patch because I'm not entirely certain it's the
> right solution to this problem.  Can you try the CVS version and fix
> against that?

In t/08_errors.t:

Looks like cygwin supports test 22 but not test 21... the fix is probably
more like this:

    # Odd parameters.
    if ($^O eq 'cgwin') {
      print "ok 21 # skipped: $^O does not support listen on unbound
sockets.\n";
    }
    else {
      stderr_pause();
      POE::Wheel::SocketFactory->new
        ( SuccessEvent => [ ],
          FailureEvent => [ ],
        );
      stderr_resume();

      print "not " unless $warnings == 2;
      print "ok 21\n";
    }

Alan 

---------------
Alan Hastings
ahastings circle-a dynamicsoft.com


Reply via email to