Mojolicious test in cygwin
36 views
Skip to first unread message
Silvio
Jul 13, 2011, 2:29:27 PM7/13/11
to Mojolicious
Hi friends.
I tested Mojolicious in cygwin environment, and some error occurs on
ioloop test. Basically, in the last test (error test), the program not
drops the connection in lines 131 - 132, and waits indefinitily.
I solve this patching Blocking connection ...
$ git diff
diff --git a/lib/Mojo/IOLoop.pm b/lib/Mojo/IOLoop.pm
index 1f8ab14..28b54b1 100644
--- a/lib/Mojo/IOLoop.pm
+++ b/lib/Mojo/IOLoop.pm
@@ -585,7 +585,7 @@ sub _connect {
# New socket
my %options = (
- Blocking => 0,
+ Blocking => WINDOWS ? 1 : 0,
PeerAddr => $args->{address},
PeerPort => $args->{port} || ($args->{tls} ? 443 : 80),
Proto => $args->{proto},
... but, I think is a poor solution.
There are another (better) way to solve?
Silvio
I tested Mojolicious in cygwin environment, and some error occurs on
ioloop test. Basically, in the last test (error test), the program not
drops the connection in lines 131 - 132, and waits indefinitily.
I solve this patching Blocking connection ...
$ git diff
diff --git a/lib/Mojo/IOLoop.pm b/lib/Mojo/IOLoop.pm
index 1f8ab14..28b54b1 100644
--- a/lib/Mojo/IOLoop.pm
+++ b/lib/Mojo/IOLoop.pm
@@ -585,7 +585,7 @@ sub _connect {
# New socket
my %options = (
- Blocking => 0,
+ Blocking => WINDOWS ? 1 : 0,
PeerAddr => $args->{address},
PeerPort => $args->{port} || ($args->{tls} ? 443 : 80),
Proto => $args->{proto},
... but, I think is a poor solution.
There are another (better) way to solve?
Silvio
sri
Jul 13, 2011, 4:27:18 PM7/13/11
to Mojolicious
If there's a bug, why doesn't it show up on cpantesters?
http://www.cpantesters.org/distro/M/Mojolicious.html
--
sebastian
http://www.cpantesters.org/distro/M/Mojolicious.html
--
sebastian
Silvio
Jul 13, 2011, 10:36:54 PM7/13/11
to Mojolicious
I think is not Mojolicious bug, it's a Windows bug !
Well, I tried to send a cpantester report, but, the test never ends. I
don't know (and I don't found in google) if there are a option for
cancel the test by timeout and send the report.
$ perl -v
This is perl, v5.10.1 (*) built for i686-cygwin-thread-multi-64int
(with 13 registered patches, see perl -V for more detail)
Copyright 1987-2009, Larry Wall
Perl may be copied only under the terms of either the Artistic License
or the
GNU General Public License, which may be found in the Perl 5 source
kit.
Complete documentation for Perl, including FAQ lists, should be found
on
this system using "man perl" or "perldoc perl". If you have access to
the
Internet, point your browser at http://www.perl.org/, the Perl Home
Page.
sri
Jul 13, 2011, 10:41:29 PM7/13/11
to Mojolicious
Afraid i won't be of much help here, since i know pretty much nothing
about Windows.
But such a harsh change can't be done without confirmation from
multiple sources and more information about why exactly it is
necessary.
--
sebastian
about Windows.
But such a harsh change can't be done without confirmation from
multiple sources and more information about why exactly it is
necessary.
--
sebastian
fibo
Jul 14, 2011, 11:28:15 AM7/14/11
to Mojolicious
Hi Silvio,
are you obligated by some policy or something to use cygwin ?
Otherwise I suggest you to switch to Strawberry Perl distribution.
are you obligated by some policy or something to use cygwin ?
Otherwise I suggest you to switch to Strawberry Perl distribution.
Silvio
Jul 15, 2011, 8:39:54 AM7/15/11
to Mojolicious
with cygwin ... I'll test another thing in the weekend and I'll write
again if I succeed better solution.
Silvio
Reply all
Reply to author
Forward
0 new messages