Bug #78673 for Devel-REPL: Lexical::Persistence not optional?

Mon Jul 30 12:55:32 2012 TODDR [...] cpan.org - Ticket created

Subject:

Lexical::Persistence not optional?

Running the test suite against the latest version, I notice that without Lexical::Persistence, it will not pass. This is listed as an optional feature. Perhaps it should not be?

Mon Aug 27 04:17:11 2012 d.thomas [...] its.uq.edu.au - Correspondence added

From:

d.thomas [...] its.uq.edu.au

On Mon Jul 30 12:55:32 2012, TODDR wrote: Show quoted text

> Running the test suite against the latest version, I notice that > without Lexical::Persistence, it will > not pass. This is listed as an optional feature. Perhaps it should not be?

+1 The automated build system I use only installs when tests PASS Devel-REPL-1.003013.tar.gz 39 PERL_DL_NONLAZY=1 /opt/perl/uq.cis.perl-dmt-toys.rhel5-5.16.1-20120825/bin/perl "- MExtUtils::Command::MM" "-e" "test_harness(0, 'inc', 'blib/lib', 'blib/arch')" t/load_core.t t/load_core.t .. 1/? # Failed test 'use Devel::REPL::Plugin::LexEnv;' # at t/load_core.t line 54. # Tried to use 'Devel::REPL::Plugin::LexEnv'. # Error: Can't locate Lexical/Persistence.pm t/load_core.t (Wstat: 256 Tests: 23 Failed: 1) Failed test: 14 Non-zero exit status: 1 Files=1, Tests=23, 1 wallclock secs ( 0.02 usr 0.02 sys + 0.78 cusr 0.06 csys = 0.88 CPU) Result: FAIL Failed 1/1 test programs. 1/23 subtests failed. Including Lexical-Persistence-1.020.tar.gz in the tarball list says it build, test and install

Mon Aug 27 04:17:14 2012 The RT System itself - Status changed from 'new' to 'open'

Mon Aug 27 06:59:24 2012 chm [...] cpan.org - Correspondence added

Thanks for investigating this issue. At the least, our
test suite should not fail if an optional feature is not
enabled. I think the current tests just check whether
things load or not.

--Chris

On Mon Aug 27 04:17:11 2012, d.thomas@its.uq.edu.au wrote:
Show quoted text

> On Mon Jul 30 12:55:32 2012, TODDR wrote:

> > Running the test suite against the latest version, I notice that
> > without Lexical::Persistence, it will
> > not pass. This is listed as an optional feature. Perhaps it should

> not be?
>
> +1
>
> The automated build system I use only installs when tests PASS
>
> Devel-REPL-1.003013.tar.gz 39
> PERL_DL_NONLAZY=1 /opt/perl/uq.cis.perl-dmt-toys.rhel5-5.16.1-
> 20120825/bin/perl "-
> MExtUtils::Command::MM" "-e" "test_harness(0, 'inc', 'blib/lib',
> 'blib/arch')" t/load_core.t
> t/load_core.t .. 1/?

> # Failed test 'use Devel::REPL::Plugin::LexEnv;'
> # at t/load_core.t line 54.
> # Tried to use 'Devel::REPL::Plugin::LexEnv'.
> # Error: Can't locate Lexical/Persistence.pm

> t/load_core.t (Wstat: 256 Tests: 23 Failed: 1)
> Failed test: 14
> Non-zero exit status: 1
> Files=1, Tests=23, 1 wallclock secs ( 0.02 usr 0.02 sys + 0.78 cusr
> 0.06 csys = 0.88 CPU)
> Result: FAIL
> Failed 1/1 test programs. 1/23 subtests failed.
>
>
> Including Lexical-Persistence-1.020.tar.gz in the tarball list
> says it build, test and install

Fri Mar 01 13:29:42 2013 ether [...] cpan.org - Correspondence added

Fixed in 1.003015.

Fri Mar 01 13:29:45 2013 ether [...] cpan.org - Status changed from 'open' to 'resolved'