Hi,

- the t/mojolicious/websocket_lite_app.t never passed w/o warnings

- TEST_IPV6=Yes requires an IPV6 environment of course

- the test certificate shouldn't reside unter ${PREFIX}/${P5SITE}/,
  that's why i added the README and dropped it from the pkg.


Besides that i'm ok with it. Seems like very few people are using
p5-Mojo, but...it's definitely time to get this updated.


On Sat, Nov 04, 2017 at 01:42:00PM +0100, Remi Locherer wrote:
> Hi,
> 
> This is an update for Mojolicious. Lots of changes since 6.24, see
> https://github.com/kraih/mojo/blob/v7.52/Changes.
> 
> There was a discussion about updating this port earlier this year:
> https://marc.info/?t=148629827100001&r=1&w=2
> 
> I think the reasons for holding it back are not present anymore.
> 
> The tests pass but with some warnings:
> 
> ---%<---
> t/mojolicious/validation_lite_app.t ........ ok
> Use of uninitialized value in numeric ge (>=) at 
> /usr/ports/pobj/p5-Mojolicious-7.52/Mojolicious-7.52/blib/lib/Mojo/Server/Daemon.pm
>  line 109.
> Use of uninitialized value $loop in hash element at 
> /usr/ports/pobj/p5-Mojolicious-7.52/Mojolicious-7.52/blib/lib/Mojo/UserAgent.pm
>  line 220.
> Mojo::Reactor::Poll: I/O watcher failed: Can't call method "remove" on an 
> undefined value at 
> /usr/ports/pobj/p5-Mojolicious-7.52/Mojolicious-7.52/blib/lib/Mojo/UserAgent.pm
>  line 296.
> t/mojolicious/websocket_lite_app.t ......... ok
> t/pod.t .................................... ok
> t/pod_coverage.t ........................... ok
> All tests successful.
> Files=96, Tests=11556, 137 wallclock secs ( 2.96 usr  1.03 sys + 64.29 cusr 
> 11.32 csys = 79.60 CPU)
> Result: PASS
> ---%<---
> 
> Test of www/p5-MojoX-Session pass.
> 
> I'll send and update for www/p5-Mojolicious-Plugin-TtRenderer with the
> next mail.
> 
> OK?
> 
> Remi
> 
> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/www/p5-Mojo/Makefile,v
> retrieving revision 1.28
> diff -u -p -r1.28 Makefile
> --- Makefile  20 Mar 2016 19:57:21 -0000      1.28
> +++ Makefile  4 Nov 2017 12:21:59 -0000
> @@ -4,7 +4,7 @@ COMMENT =     next generation web framework 
>  
>  MODULES =    cpan
>  PKG_ARCH =   *
> -DISTNAME =   Mojolicious-6.24
> +DISTNAME =   Mojolicious-7.52
>  CATEGORIES = www
>  
>  MAINTAINER = Mark Patruck <m...@wrapped.cx>
> @@ -19,15 +19,17 @@ CPAN_AUTHOR =     SRI
>  RUN_DEPENDS =        devel/p5-EV \
>               security/p5-IO-Socket-SSL>=1.94
>  
> -MAKE_ENV += TEST_CACHING=Yes
> +TEST_DEPENDS +=      net/p5-IO-Socket-Socks
> +
>  MAKE_ENV += TEST_EV=Yes
>  MAKE_ENV += TEST_HYPNOTOAD=Yes
> +MAKE_ENV += TEST_IPV6=Yes
>  MAKE_ENV += TEST_MORBO=Yes
>  MAKE_ENV += TEST_POD=Yes
>  MAKE_ENV += TEST_PREFORK=Yes
> -
> -# see pkg/README for certificate handling
> -post-install:
> -     rm -rf ${PREFIX}/${P5SITE}/Mojo/IOLoop/certs/
> +MAKE_ENV += TEST_SOCKS=Yes
> +MAKE_ENV += TEST_SUBPROCESS=Yes
> +MAKE_ENV += TEST_TLS=Yes
> +MAKE_ENV += TEST_UNIX=YES
>  
>  .include <bsd.port.mk>
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/www/p5-Mojo/distinfo,v
> retrieving revision 1.21
> diff -u -p -r1.21 distinfo
> --- distinfo  15 Oct 2015 21:01:06 -0000      1.21
> +++ distinfo  4 Nov 2017 12:21:59 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (Mojolicious-6.24.tar.gz) = 
> bgEog361mgykZJuUjP/i1WVCiYur2fffAxesY5pFEiU=
> -SIZE (Mojolicious-6.24.tar.gz) = 651857
> +SHA256 (Mojolicious-7.52.tar.gz) = 
> yohMIKqM4qAi+w4/AhdWCepeWM/uty1h/YQEb9vINso=
> +SIZE (Mojolicious-7.52.tar.gz) = 695680
> Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/www/p5-Mojo/pkg/PLIST,v
> retrieving revision 1.10
> diff -u -p -r1.10 PLIST
> --- pkg/PLIST 19 Jul 2015 14:23:45 -0000      1.10
> +++ pkg/PLIST 4 Nov 2017 12:21:59 -0000
> @@ -28,6 +28,7 @@ ${P5SITE}/Mojo/DOM/HTML.pm
>  ${P5SITE}/Mojo/Date.pm
>  ${P5SITE}/Mojo/EventEmitter.pm
>  ${P5SITE}/Mojo/Exception.pm
> +${P5SITE}/Mojo/File.pm
>  ${P5SITE}/Mojo/Headers.pm
>  ${P5SITE}/Mojo/HelloWorld.pm
>  ${P5SITE}/Mojo/Home.pm
> @@ -37,6 +38,11 @@ ${P5SITE}/Mojo/IOLoop/Client.pm
>  ${P5SITE}/Mojo/IOLoop/Delay.pm
>  ${P5SITE}/Mojo/IOLoop/Server.pm
>  ${P5SITE}/Mojo/IOLoop/Stream.pm
> +${P5SITE}/Mojo/IOLoop/Subprocess.pm
> +${P5SITE}/Mojo/IOLoop/TLS.pm
> +${P5SITE}/Mojo/IOLoop/resources/
> +${P5SITE}/Mojo/IOLoop/resources/server.crt
> +${P5SITE}/Mojo/IOLoop/resources/server.key
>  ${P5SITE}/Mojo/JSON/
>  ${P5SITE}/Mojo/JSON.pm
>  ${P5SITE}/Mojo/JSON/Pointer.pm
> @@ -57,7 +63,11 @@ ${P5SITE}/Mojo/Server.pm
>  ${P5SITE}/Mojo/Server/CGI.pm
>  ${P5SITE}/Mojo/Server/Daemon.pm
>  ${P5SITE}/Mojo/Server/Hypnotoad.pm
> +${P5SITE}/Mojo/Server/Morbo/
>  ${P5SITE}/Mojo/Server/Morbo.pm
> +${P5SITE}/Mojo/Server/Morbo/Backend/
> +${P5SITE}/Mojo/Server/Morbo/Backend.pm
> +${P5SITE}/Mojo/Server/Morbo/Backend/Poll.pm
>  ${P5SITE}/Mojo/Server/PSGI.pm
>  ${P5SITE}/Mojo/Server/Prefork.pm
>  ${P5SITE}/Mojo/Template.pm
> @@ -74,6 +84,7 @@ ${P5SITE}/Mojo/UserAgent/Proxy.pm
>  ${P5SITE}/Mojo/UserAgent/Server.pm
>  ${P5SITE}/Mojo/UserAgent/Transactor.pm
>  ${P5SITE}/Mojo/Util.pm
> +${P5SITE}/Mojo/WebSocket.pm
>  ${P5SITE}/Mojolicious/
>  ${P5SITE}/Mojolicious.pm
>  ${P5SITE}/Mojolicious/Command/
> @@ -105,11 +116,11 @@ ${P5SITE}/Mojolicious/Guides/FAQ.pod
>  ${P5SITE}/Mojolicious/Guides/Growing.pod
>  ${P5SITE}/Mojolicious/Guides/Rendering.pod
>  ${P5SITE}/Mojolicious/Guides/Routing.pod
> +${P5SITE}/Mojolicious/Guides/Testing.pod
>  ${P5SITE}/Mojolicious/Guides/Tutorial.pod
>  ${P5SITE}/Mojolicious/Lite.pm
>  ${P5SITE}/Mojolicious/Plugin/
>  ${P5SITE}/Mojolicious/Plugin.pm
> -${P5SITE}/Mojolicious/Plugin/Charset.pm
>  ${P5SITE}/Mojolicious/Plugin/Config.pm
>  ${P5SITE}/Mojolicious/Plugin/DefaultHelpers.pm
>  ${P5SITE}/Mojolicious/Plugin/EPLRenderer.pm
> @@ -132,29 +143,31 @@ ${P5SITE}/Mojolicious/Types.pm
>  ${P5SITE}/Mojolicious/Validator/
>  ${P5SITE}/Mojolicious/Validator.pm
>  ${P5SITE}/Mojolicious/Validator/Validation.pm
> -${P5SITE}/Mojolicious/public/
> -${P5SITE}/Mojolicious/public/favicon.ico
> -${P5SITE}/Mojolicious/public/mojo/
> -${P5SITE}/Mojolicious/public/mojo/failraptor.png
> -${P5SITE}/Mojolicious/public/mojo/jquery/
> -${P5SITE}/Mojolicious/public/mojo/jquery/jquery.js
> -${P5SITE}/Mojolicious/public/mojo/logo-black.png
> -${P5SITE}/Mojolicious/public/mojo/logo-white.png
> -${P5SITE}/Mojolicious/public/mojo/noraptor.png
> -${P5SITE}/Mojolicious/public/mojo/notfound.png
> -${P5SITE}/Mojolicious/public/mojo/pinstripe-dark.png
> -${P5SITE}/Mojolicious/public/mojo/pinstripe-light.png
> -${P5SITE}/Mojolicious/public/mojo/prettify/
> -${P5SITE}/Mojolicious/public/mojo/prettify/prettify-mojo-dark.css
> -${P5SITE}/Mojolicious/public/mojo/prettify/prettify-mojo-light.css
> -${P5SITE}/Mojolicious/public/mojo/prettify/run_prettify.js
> -${P5SITE}/Mojolicious/public/mojo/stripes.png
> -${P5SITE}/Mojolicious/templates/
> -${P5SITE}/Mojolicious/templates/development.html.ep
> -${P5SITE}/Mojolicious/templates/exception.html.ep
> -${P5SITE}/Mojolicious/templates/mojobar.html.ep
> -${P5SITE}/Mojolicious/templates/not_found.html.ep
> -${P5SITE}/Mojolicious/templates/perldoc.html.ep
> +${P5SITE}/Mojolicious/resources/
> +${P5SITE}/Mojolicious/resources/public/
> +${P5SITE}/Mojolicious/resources/public/favicon.ico
> +${P5SITE}/Mojolicious/resources/public/mojo/
> +${P5SITE}/Mojolicious/resources/public/mojo/failraptor.png
> +${P5SITE}/Mojolicious/resources/public/mojo/jquery/
> +${P5SITE}/Mojolicious/resources/public/mojo/jquery/jquery.js
> +${P5SITE}/Mojolicious/resources/public/mojo/logo-black.png
> +${P5SITE}/Mojolicious/resources/public/mojo/logo-white.png
> +${P5SITE}/Mojolicious/resources/public/mojo/noraptor.png
> +${P5SITE}/Mojolicious/resources/public/mojo/notfound.png
> +${P5SITE}/Mojolicious/resources/public/mojo/pinstripe-dark.png
> +${P5SITE}/Mojolicious/resources/public/mojo/pinstripe-light.png
> +${P5SITE}/Mojolicious/resources/public/mojo/prettify/
> +${P5SITE}/Mojolicious/resources/public/mojo/prettify/prettify-mojo-dark.css
> +${P5SITE}/Mojolicious/resources/public/mojo/prettify/prettify-mojo-light.css
> +${P5SITE}/Mojolicious/resources/public/mojo/prettify/run_prettify.js
> +${P5SITE}/Mojolicious/resources/public/mojo/stripes.png
> +${P5SITE}/Mojolicious/resources/templates/
> +${P5SITE}/Mojolicious/resources/templates/mojo/
> +${P5SITE}/Mojolicious/resources/templates/mojo/debug.html.ep
> +${P5SITE}/Mojolicious/resources/templates/mojo/exception.html.ep
> +${P5SITE}/Mojolicious/resources/templates/mojo/menubar.html.ep
> +${P5SITE}/Mojolicious/resources/templates/mojo/not_found.html.ep
> +${P5SITE}/Mojolicious/resources/templates/mojo/perldoc.html.ep
>  ${P5SITE}/Test/
>  ${P5SITE}/Test/Mojo.pm
>  ${P5SITE}/ojo.pm
> @@ -181,6 +194,7 @@ ${P5SITE}/ojo.pm
>  @man man/man3p/Mojo::Date.3p
>  @man man/man3p/Mojo::EventEmitter.3p
>  @man man/man3p/Mojo::Exception.3p
> +@man man/man3p/Mojo::File.3p
>  @man man/man3p/Mojo::Headers.3p
>  @man man/man3p/Mojo::HelloWorld.3p
>  @man man/man3p/Mojo::Home.3p
> @@ -189,6 +203,8 @@ ${P5SITE}/ojo.pm
>  @man man/man3p/Mojo::IOLoop::Delay.3p
>  @man man/man3p/Mojo::IOLoop::Server.3p
>  @man man/man3p/Mojo::IOLoop::Stream.3p
> +@man man/man3p/Mojo::IOLoop::Subprocess.3p
> +@man man/man3p/Mojo::IOLoop::TLS.3p
>  @man man/man3p/Mojo::JSON.3p
>  @man man/man3p/Mojo::JSON::Pointer.3p
>  @man man/man3p/Mojo::Loader.3p
> @@ -206,6 +222,8 @@ ${P5SITE}/ojo.pm
>  @man man/man3p/Mojo::Server::Daemon.3p
>  @man man/man3p/Mojo::Server::Hypnotoad.3p
>  @man man/man3p/Mojo::Server::Morbo.3p
> +@man man/man3p/Mojo::Server::Morbo::Backend.3p
> +@man man/man3p/Mojo::Server::Morbo::Backend::Poll.3p
>  @man man/man3p/Mojo::Server::PSGI.3p
>  @man man/man3p/Mojo::Server::Prefork.3p
>  @man man/man3p/Mojo::Template.3p
> @@ -220,6 +238,7 @@ ${P5SITE}/ojo.pm
>  @man man/man3p/Mojo::UserAgent::Server.3p
>  @man man/man3p/Mojo::UserAgent::Transactor.3p
>  @man man/man3p/Mojo::Util.3p
> +@man man/man3p/Mojo::WebSocket.3p
>  @man man/man3p/Mojolicious.3p
>  @man man/man3p/Mojolicious::Command.3p
>  @man man/man3p/Mojolicious::Command::cgi.3p
> @@ -247,10 +266,10 @@ ${P5SITE}/ojo.pm
>  @man man/man3p/Mojolicious::Guides::Growing.3p
>  @man man/man3p/Mojolicious::Guides::Rendering.3p
>  @man man/man3p/Mojolicious::Guides::Routing.3p
> +@man man/man3p/Mojolicious::Guides::Testing.3p
>  @man man/man3p/Mojolicious::Guides::Tutorial.3p
>  @man man/man3p/Mojolicious::Lite.3p
>  @man man/man3p/Mojolicious::Plugin.3p
> -@man man/man3p/Mojolicious::Plugin::Charset.3p
>  @man man/man3p/Mojolicious::Plugin::Config.3p
>  @man man/man3p/Mojolicious::Plugin::DefaultHelpers.3p
>  @man man/man3p/Mojolicious::Plugin::EPLRenderer.3p
> 

-- 
Mark Patruck ( mark at wrapped.cx )
GPG key 0xF2865E51 / 187F F6D3 EE04 1DCE 1C74  F644 0D3C F66F F286 5E51

http://www.wrapped.cx

Reply via email to