Bug #78333 for Mojolicious-Plugin-HamlRenderer: Fast growing process in testsuite

Thu Jul 12 15:59:23 2012 ANDK [...] cpan.org - Ticket created

Subject:

Fast growing process in testsuite

% ps auxww [...] sand 14084 98.8 54.4 9006968 8872552 ? RN 21:30 0:41 /mnt/k75/homesand/src/perl/repoperls/installed-perls/perl/v5.17.1-343-gd5f981b/9980/bin/perl t/lite_app.t The process reaches this size in just a few seconds. When I run the test script in the debugger the growing starts in this line: 218: return 1 if $renderer->($self, $c, $output, $options); I hit ^C at this point and the debugger was at: Mojo::Base::CODE(0x38620c0)((eval 199)[/home/src/perl/repoperls/installed-perls/perl/v5.17.1-343-gd5f981b/9980/lib/site_perl/5.17.2/Mojo/Base.pm:97]:3): 3: return $_[0]{'helpers'} if exists $_[0]{'helpers'}; If you cannot reproduce, please ask me some questions. I pobably won't be online much in the next two weeks though due to holiday season. I see that I sent you success reports, e.g. http://www.cpantesters.org/cpan/report/8909d1a4-c567-11e1-aca9-ec7081a2c3a6. The success was with v5.17.1-289-g5e5bb7c while the explosion was with v5.17.1-343-gd5f981b, so maybe it is a bleadperl issue. Will try to find some more evidence ASAP.

Thu Jul 12 23:56:36 2012 ANDK [...] cpan.org - Correspondence added

It's not a bleadperl issue. After I upgraded everything in my installation of v5.17.1-289-g5e5bb7c I could reproduce the issue with that perl too. So some dependency is to blame.

Thu Jul 12 23:56:37 2012 ANDK [...] cpan.org - Status changed from 'new' to 'open'

Fri Jul 13 08:16:02 2012 VTI [...] cpan.org - Correspondence added

On Thu Jul 12 23:56:36 2012, ANDK wrote: Show quoted text

> It's not a bleadperl issue. After I upgraded everything in my > installation of v5.17.1-289-g5e5bb7c I could reproduce the issue with > that perl too. So some dependency is to blame.

Thanks for the report. I tried to bump Mojolicious version. But that requires a lot of rewrite. I don't have time and will to do so. I am looking for a new maintainer of this module. If nobody wants to do that, I'll just remove the distribution from CPAN.

Mon Jul 30 11:23:18 2012 garu [...] cpan.org - Correspondence added

On Thu Jul 12 23:56:36 2012, ANDK wrote: Show quoted text

> It's not a bleadperl issue. After I upgraded everything in my > installation of v5.17.1-289-g5e5bb7c I could reproduce the issue with > that perl too. So some dependency is to blame.

Hey Andreas, could you help me reproduce the issue? I'm using: * perl 5.17.2 (installed via perlbrew) * Mojolicious 3.15 * Text::Haml 0.990104 The CPAN version of MojoX::Renderer::Haml wasn't working with Mojolicious > 1.13, so I had to tweak it a bit. I'm using the version in: https://github.com/garu/mojox-renderer-haml quick download link: https://github.com/garu/mojox-renderer-haml/tarball/master Thanks!

Mon Jul 30 16:29:51 2012 andreas.koenig.7os6VVqR [...] franz.ak.mind.de - Correspondence added

CC:	ANDK [...] cpan.org
Subject:	Re: [rt.cpan.org #78333] Fast growing process in testsuite
Date:	Mon, 30 Jul 2012 22:29:34 +0200
To:	bug-Mojolicious-Plugin-HamlRenderer [...] rt.cpan.org
From:	andreas.koenig.7os6VVqR [...] franz.ak.mind.de (Andreas J. Koenig)

Show quoted text

>>>>> On Mon, 30 Jul 2012 11:23:18 -0400, "Breno G. de Oliveira via RT" <bug-Mojolicious-Plugin-HamlRenderer@rt.cpan.org> said:

Show quoted text

> could you help me reproduce the issue?

Sure, let me try... Show quoted text

> I'm using:

Show quoted text

> * perl 5.17.2 (installed via perlbrew)

Trying with v5.17.2-6-gf1f8fd6 built traditionally. Show quoted text

> * Mojolicious 3.15 > * Text::Haml 0.990104

OK, same here. Show quoted text

> The CPAN version of MojoX::Renderer::Haml wasn't working with > Mojolicious > 1.13, so I had to tweak it a bit. I'm using the version in:

Show quoted text

> https://github.com/garu/mojox-renderer-haml

Show quoted text

> quick download link: > https://github.com/garu/mojox-renderer-haml/tarball/master

Downloaded and untarred gives directory garu-mojox-renderer-haml-4170c8f. Show quoted text

>sand@k83:/tmp/garu-mojox-renderer-haml-4170c8f% /home/sand/src/perl/repoperls/installed-perls/perl/v5.17.2-6-g*/*/bin/perl Makefile.PL

Writing Makefile for MojoX::Renderer::Haml Writing MYMETA.yml and MYMETA.json Show quoted text

>sand@k83:/tmp/garu-mojox-renderer-haml-4170c8f% make

cp lib/Mojolicious/Plugin/HamlRenderer.pm blib/lib/Mojolicious/Plugin/HamlRenderer.pm cp lib/MojoX/Renderer/Haml.pm blib/lib/MojoX/Renderer/Haml.pm Manifying blib/man3/Mojolicious::Plugin::HamlRenderer.3 Manifying blib/man3/MojoX::Renderer::Haml.3 Show quoted text

>sand@k83:/tmp/garu-mojox-renderer-haml-4170c8f% make test

PERL_DL_NONLAZY=1 /mnt/k75/homesand/src/perl/repoperls/installed-perls/perl/v5.17.2-6-gf1f8fd6/165a/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/*/*.t t/*/*/*.t t/*/*/*/*.t t/00load.t .... Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/^(.*?)?(?<!\\)\#{ <-- HERE / at /home/src/perl/repoperls/installed-perls/perl/v5.17.2-6-gf1f8fd6/165a/lib/site_perl/5.17.3/Text/Haml.pm line 855. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/^(.*?)?\\\\\#{ <-- HERE / at /home/src/perl/repoperls/installed-perls/perl/v5.17.2-6-gf1f8fd6/165a/lib/site_perl/5.17.3/Text/Haml.pm line 859. t/00load.t .... ok t/lite_app.t .. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/^(.*?)?(?<!\\)\#{ <-- HERE / at /home/src/perl/repoperls/installed-perls/perl/v5.17.2-6-gf1f8fd6/165a/lib/site_perl/5.17.3/Text/Haml.pm line 855. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/^(.*?)?\\\\\#{ <-- HERE / at /home/src/perl/repoperls/installed-perls/perl/v5.17.2-6-gf1f8fd6/165a/lib/site_perl/5.17.3/Text/Haml.pm line 859. t/lite_app.t .. ok All tests successful. Files=2, Tests=19, 0 wallclock secs ( 0.02 usr 0.00 sys + 0.35 cusr 0.03 csys = 0.40 CPU) Result: PASS Show quoted text

> Thanks!

You're welcome! Thanks for fixing! -- andreas

Tue Aug 14 10:29:35 2012 garu [...] cpan.org - Taken

Tue Aug 14 10:30:22 2012 garu [...] cpan.org - Correspondence added

New release should hit CPAN in a few minutes!

Tue Aug 14 10:30:23 2012 garu [...] cpan.org - Status changed from 'open' to 'resolved'