Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the config-model CPAN distribution.

Report information
The Basics
Id: 74900
Status: resolved
Priority: 0/
Queue: config-model
People
Owner: DDUMONT [...] cpan.org
Requestors: ANDK [...] cpan.org
Cc: sprout [...] cpan.org
AdminCc:
Bug Information
Severity: Normal
Broken in:
  • 2.004
  • 2.005
  • 2.006
  • 2.007
  • 2.008
  • 2.009
  • 2.010
  • 2.011
  • 2.012
Fixed in: 2.013

History Show all quoted text
  Sun Feb 12 17:03:49 2012 ANDK [...] cpan.org - Ticket created
CC: sprout [...] cpan.org
Subject: Fails since bleadperl v5.14.0-319-g8fe85e3
A bisect of DDUMONT/Config-Model-2.004.tar.gz test runs against bleadperl ends at commit 8fe85e3fe48130bc5b95139890db97fc7a6c2c49 Author: Father Chrysostomos <sprout@cpan.org> Date: Mon May 30 09:49:36 2011 -0700 [perl #31946] Warn when assigning to a TEMP Sample fail report: http://www.cpantesters.org/cpan/report/b7a3c42a-556e-11e1-9731-ed709aeef8c6 Text::Balanced is 2.02 in case it matters. HTH&&Regards,
  Mon Feb 20 07:11:41 2012 DDUMONT [...] cpan.org - Correspondence added
On Sun Feb 12 23:03:49 2012, ANDK wrote: Show quoted text
> [perl #31946] Warn when assigning to a TEMP > > Sample fail report: > >
http://www.cpantesters.org/cpan/report/b7a3c42a-556e-11e1-9731-ed709aeef8c6 Show quoted text
> > Text::Balanced is 2.02 in case it matters.
I've the same version of Text::Balanced. Config::Model does not use directly Text::Balanced. IIRC, Parse::RecDescent uses Text::Balanced. I'm not sure what I can do at this point ... All the best
  Mon Feb 20 07:11:42 2012 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 21 00:12:42 2012 ANDK [...] cpan.org - Correspondence added
RT-Send-CC: ADAMK [...] cpan.org, DCONWAY [...] cpan.org
Are you saying Config::Model's tests are triggering a bug in either Parse::RecDescent or Text::Balanced while the tests of P:RD and T:B do not trigger that bug? This sounds like the test case for the other two modules needs to be isolated and must be brought to their maintainers' attention. CC'ing to the respective maintainers with a plea to increase their level of attention to this case.
  Wed Feb 22 04:27:24 2012 DDUMONT [...] cpan.org - Correspondence added
On Tue Feb 21 06:12:42 2012, ANDK wrote: Show quoted text
> Are you saying Config::Model's tests are triggering a bug in either > Parse::RecDescent or Text::Balanced while the tests of P:RD and T:B do > not trigger that bug?
As a matter of fact, the test that produces all those warnings (t/accept.t) loads Parse::RecDescent without using it. So I think that Parse::RecDescent or Text::Balanced have an issue with bleadperl (at load time) but are not yet tested with a recent enough version of bleadperl Or bleadperl has a bug. Hope this helps
  Sun Feb 26 13:59:37 2012 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added
On Wed Feb 22 04:27:24 2012, DDUMONT wrote: Show quoted text
> Or bleadperl has a bug.
The new bleadperl warning reveals an existing Mouse bug. See #75313.
  Mon Mar 05 08:29:11 2012 ANDK [...] cpan.org - Broken in 2.005 added
  Mon Mar 05 08:29:11 2012 ANDK [...] cpan.org - Broken in 2.006 added
  Mon Mar 05 08:29:11 2012 ANDK [...] cpan.org - Broken in 2.007 added
  Mon Mar 05 08:29:11 2012 ANDK [...] cpan.org - Broken in 2.008 added
  Tue Apr 03 01:12:07 2012 ANDK [...] cpan.org - Broken in 2.009 added
  Tue Apr 03 01:12:07 2012 ANDK [...] cpan.org - Broken in 2.010 added
  Tue Apr 03 01:12:07 2012 ANDK [...] cpan.org - Broken in 2.011 added
  Thu Apr 05 07:54:21 2012 DDUMONT [...] cpan.org - Correspondence added
On Sun Feb 26 19:59:37 2012, SPROUT wrote: Show quoted text
> On Wed Feb 22 04:27:24 2012, DDUMONT wrote:
> > Or bleadperl has a bug.
> > The new bleadperl warning reveals an existing Mouse bug. See #75313.
I've been able to reproduce the test problem thanks Xavier Caron's advices during Perl hackathon. The test failed due to a warning during multistrap model test. This warning is harmless so the test just ignore it now. This test problem is unrelated to the Text:Balanced warning. This should be fixed in Config::Model 2.012 Thanks for the report.
  Thu Apr 05 07:54:22 2012 DDUMONT [...] cpan.org - Status changed from 'open' to 'patched'
  Thu Apr 05 23:43:15 2012 ANDK [...] cpan.org - Correspondence added
2.012 still passes with v5.14.0-318-gd34a666 and fails with v5.14.0-319-g8fe85e3. Test Summary Report ------------------- t/model_tests.t (Wstat: 256 Tests: 85 Failed: 1) Failed test: 48 Non-zero exit status: 1 Files=55, Tests=1603, 30 wallclock secs ( 0.52 usr 0.13 sys + 18.97 cusr 2.04 csys = 21.66 CPU) Result: FAIL Failed 1/55 test programs. 1/1603 subtests failed. And model_tests.t ends with: # found warning: Useless assignment to a temporary at /home/src/perl/repoperls/installed-perls/perl/v5.14.0-319-g8fe85e3/a2da/lib/5.15.0/Text/Balanced.pm line 44, <GEN14> line 22. # found warning: Useless assignment to a temporary at /home/src/perl/repoperls/installed-perls/perl/v5.14.0-319-g8fe85e3/a2da/lib/5.15.0/Text/Balanced.pm line 614, <GEN14> line 22. # found warning: Useless assignment to a temporary at /home/src/perl/repoperls/installed-perls/perl/v5.14.0-319-g8fe85e3/a2da/lib/5.15.0/Text/Balanced.pm line 58, <GEN14> line 22. # didn't expect to find a warning Unhandled type: GLOB at /home/src/perl/repoperls/installed-perls/perl/v5.14.0-319-g8fe85e3/a2da/lib/site_perl/5.15.0/Devel/Cycle.pm line 107. # Looks like you failed 1 test of 85. t/model_tests.t ................... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/85 subtests This "Unhandled type: GLOB" warning was also there in v5.14.0-318-gd34a666 but was ignored. HTH, Regards,
  Thu Apr 05 23:43:16 2012 The RT System itself - Status changed from 'patched' to 'open'
  Thu Apr 05 23:43:32 2012 ANDK [...] cpan.org - Broken in 2.012 added
  Fri Apr 06 13:01:02 2012 DDUMONT [...] cpan.org - Correspondence added
On Fri Apr 06 05:43:15 2012, ANDK wrote: Show quoted text
> 2.012 still passes with v5.14.0-318-gd34a666 and fails with > v5.14.0-319-g8fe85e3.
Ack. I've messed up the fix I've done during the hackathon :-( I think this issue is fixed in 2.013: the warning from Text::Balanced occured inside a warning_like section of the tests. I've changed the test to ignore these warnings (that's what's I tried to do last week end). I'll wait a few days before (hopefully) closing this ticket. Thanks
  Fri Apr 06 13:01:11 2012 DDUMONT [...] cpan.org - Taken
  Sat Apr 07 02:51:24 2012 ANDK [...] cpan.org - Correspondence added
Congrats! 2.013 works with bleadperl.
  Sat Apr 07 02:51:26 2012 ANDK [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Apr 07 02:51:49 2012 ANDK [...] cpan.org - Fixed in 2.013 added