Bug #108678 for Text-Indent: Negative repeat count does nothing

Tue Nov 10 14:55:08 2015 SREZIC [...] cpan.org - Ticket created

Subject:

Negative repeat count does nothing

With recent perls (e.g. 5.22.1-RC1) there's a new warning causing a test failure: # Failed test 'no warnings' # at /opt/perl-5.22.1-RC1/lib/site_perl/5.22.1/Test/NoWarnings.pm line 45. # There were 1 warning(s) # Previous test 7 'reset indent level' # Negative repeat count does nothing at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02-CTVop8/blib/lib/Text/Indent.pm line 303. # at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02-CTVop8/blib/lib/Text/Indent.pm line 303. # Text::Indent::indent(Text::Indent=HASH(0x27a7160), "foo") called at t/09_indent.t line 31 # # Looks like you failed 1 test of 9. t/09_indent.t ................. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/9 subtests

Tue Nov 10 14:57:38 2015 SREZIC [...] cpan.org - Correspondence added

On 2015-11-10 14:55:08, SREZIC wrote: Show quoted text

> With recent perls (e.g. 5.22.1-RC1) there's a new warning causing a > test failure: > > # Failed test 'no warnings' > # at /opt/perl-5.22.1-RC1/lib/site_perl/5.22.1/Test/NoWarnings.pm > line 45. > # There were 1 warning(s) > # Previous test 7 'reset indent level' > # Negative repeat count does nothing at /tmpfs/.cpan- > build/2015111018/Text-Indent-0.02-CTVop8/blib/lib/Text/Indent.pm line > 303. > # at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02- > CTVop8/blib/lib/Text/Indent.pm line 303. > # Text::Indent::indent(Text::Indent=HASH(0x27a7160), "foo") > called at t/09_indent.t line 31 > # > # Looks like you failed 1 test of 9. > t/09_indent.t ................. > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/9 subtests

See also http://matrix.cpantesters.org/?dist=Text-Indent%200.02 for a fail/pass overview --- tests start failing with perl 5.22.0.

Tue Nov 10 14:57:51 2015 SREZIC [...] cpan.org - Broken in 0.02 added

Tue Nov 10 14:57:52 2015 SREZIC [...] cpan.org - Broken in 0.1 deleted

Sat Dec 15 22:11:32 2018 blaine [...] renderorange.com - Correspondence added

On Tue Nov 10 14:57:38 2015, SREZIC wrote: Show quoted text

> On 2015-11-10 14:55:08, SREZIC wrote:

> > With recent perls (e.g. 5.22.1-RC1) there's a new warning causing a > > test failure: > > > > # Failed test 'no warnings' > > # at /opt/perl-5.22.1-RC1/lib/site_perl/5.22.1/Test/NoWarnings.pm > > line 45. > > # There were 1 warning(s) > > # Previous test 7 'reset indent level' > > # Negative repeat count does nothing at /tmpfs/.cpan- > > build/2015111018/Text-Indent-0.02-CTVop8/blib/lib/Text/Indent.pm line > > 303. > > # at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02- > > CTVop8/blib/lib/Text/Indent.pm line 303. > > # Text::Indent::indent(Text::Indent=HASH(0x27a7160), "foo") > > called at t/09_indent.t line 31 > > # > > # Looks like you failed 1 test of 9. > > t/09_indent.t ................. > > Dubious, test returned 1 (wstat 256, 0x100) > > Failed 1/9 subtests

> > See also http://matrix.cpantesters.org/?dist=Text-Indent%200.02 for a > fail/pass overview --- tests start failing with perl 5.22.0.

Hello. I've added a PULL REQUEST at https://github.com/jf647/Text-Indent/pull/1 to fix this, and have been in contact with the current maintainer regarding the patch.

Sat Dec 15 22:11:33 2018 The RT System itself - Status changed from 'new' to 'open'

Mon Dec 31 00:40:13 2018 blainem [...] cpan.org - Correspondence added

On Sat Dec 15 22:11:32 2018, blaine@renderorange.com wrote: Show quoted text

> On Tue Nov 10 14:57:38 2015, SREZIC wrote:

> > On 2015-11-10 14:55:08, SREZIC wrote:

> > > With recent perls (e.g. 5.22.1-RC1) there's a new warning causing a > > > test failure: > > > > > > # Failed test 'no warnings' > > > # at /opt/perl-5.22.1-RC1/lib/site_perl/5.22.1/Test/NoWarnings.pm > > > line 45. > > > # There were 1 warning(s) > > > # Previous test 7 'reset indent level' > > > # Negative repeat count does nothing at /tmpfs/.cpan- > > > build/2015111018/Text-Indent-0.02-CTVop8/blib/lib/Text/Indent.pm > > > line > > > 303. > > > # at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02- > > > CTVop8/blib/lib/Text/Indent.pm line 303. > > > # Text::Indent::indent(Text::Indent=HASH(0x27a7160), "foo") > > > called at t/09_indent.t line 31 > > > # > > > # Looks like you failed 1 test of 9. > > > t/09_indent.t ................. > > > Dubious, test returned 1 (wstat 256, 0x100) > > > Failed 1/9 subtests

> > > > See also http://matrix.cpantesters.org/?dist=Text-Indent%200.02 for a > > fail/pass overview --- tests start failing with perl 5.22.0.

> > > Hello. I've added a PULL REQUEST at https://github.com/jf647/Text- > Indent/pull/1 to fix this, and have been in contact with the current > maintainer regarding the patch.

Hello, I've uploaded a new version to PAUSE, 0.03, which fixes this warning and allows the tests to pass for installation. As soon as the index is updated and installation works again, I'll mark this case as resolved and close. Thanks!

Mon Dec 31 00:40:14 2018 blainem [...] cpan.org - Status changed from 'open' to 'patched'

Mon Dec 31 00:40:14 2018 blainem [...] cpan.org - Taken

Mon Dec 31 11:17:07 2018 blainem [...] cpan.org - Correspondence added

On Mon Dec 31 00:40:13 2018, BLAINEM wrote: Show quoted text

> On Sat Dec 15 22:11:32 2018, blaine@renderorange.com wrote:

> > On Tue Nov 10 14:57:38 2015, SREZIC wrote:

> > > On 2015-11-10 14:55:08, SREZIC wrote:

> > > > With recent perls (e.g. 5.22.1-RC1) there's a new warning causing > > > > a > > > > test failure: > > > > > > > > # Failed test 'no warnings' > > > > # at /opt/perl-5.22.1- > > > > RC1/lib/site_perl/5.22.1/Test/NoWarnings.pm > > > > line 45. > > > > # There were 1 warning(s) > > > > # Previous test 7 'reset indent level' > > > > # Negative repeat count does nothing at /tmpfs/.cpan- > > > > build/2015111018/Text-Indent-0.02-CTVop8/blib/lib/Text/Indent.pm > > > > line > > > > 303. > > > > # at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02- > > > > CTVop8/blib/lib/Text/Indent.pm line 303. > > > > # Text::Indent::indent(Text::Indent=HASH(0x27a7160), "foo") > > > > called at t/09_indent.t line 31 > > > > # > > > > # Looks like you failed 1 test of 9. > > > > t/09_indent.t ................. > > > > Dubious, test returned 1 (wstat 256, 0x100) > > > > Failed 1/9 subtests

> > > > > > See also http://matrix.cpantesters.org/?dist=Text-Indent%200.02 for > > > a > > > fail/pass overview --- tests start failing with perl 5.22.0.

> > > > > > Hello. I've added a PULL REQUEST at https://github.com/jf647/Text- > > Indent/pull/1 to fix this, and have been in contact with the current > > maintainer regarding the patch.

> > Hello, > > I've uploaded a new version to PAUSE, 0.03, which fixes this warning > and allows the tests to pass for installation. > > As soon as the index is updated and installation works again, I'll > mark this case as resolved and close. > > Thanks!

Index is updated, confirmed the new release on CPAN, and confirmed passing on all platforms through CPAN testers. Closing out.

Mon Dec 31 11:17:08 2018 blainem [...] cpan.org - Status changed from 'patched' to 'resolved'

Mon Dec 31 11:17:09 2018 blainem [...] cpan.org - Fixed in 0.03 added

Tue Jan 01 06:20:06 2019 SREZIC [...] cpan.org - Correspondence added

On 2018-12-31 11:17:07, BLAINEM wrote: Show quoted text

> On Mon Dec 31 00:40:13 2018, BLAINEM wrote:

> > On Sat Dec 15 22:11:32 2018, blaine@renderorange.com wrote:

> > > On Tue Nov 10 14:57:38 2015, SREZIC wrote:

> > > > On 2015-11-10 14:55:08, SREZIC wrote:

> > > > > With recent perls (e.g. 5.22.1-RC1) there's a new warning > > > > > causing > > > > > a > > > > > test failure: > > > > > > > > > > # Failed test 'no warnings' > > > > > # at /opt/perl-5.22.1- > > > > > RC1/lib/site_perl/5.22.1/Test/NoWarnings.pm > > > > > line 45. > > > > > # There were 1 warning(s) > > > > > # Previous test 7 'reset indent level' > > > > > # Negative repeat count does nothing at /tmpfs/.cpan- > > > > > build/2015111018/Text-Indent-0.02- > > > > > CTVop8/blib/lib/Text/Indent.pm > > > > > line > > > > > 303. > > > > > # at /tmpfs/.cpan-build/2015111018/Text-Indent-0.02- > > > > > CTVop8/blib/lib/Text/Indent.pm line 303. > > > > > # Text::Indent::indent(Text::Indent=HASH(0x27a7160), > > > > > "foo") > > > > > called at t/09_indent.t line 31 > > > > > # > > > > > # Looks like you failed 1 test of 9. > > > > > t/09_indent.t ................. > > > > > Dubious, test returned 1 (wstat 256, 0x100) > > > > > Failed 1/9 subtests

> > > > > > > > See also http://matrix.cpantesters.org/?dist=Text-Indent%200.02 > > > > for > > > > a > > > > fail/pass overview --- tests start failing with perl 5.22.0.

> > > > > > > > > Hello. I've added a PULL REQUEST at https://github.com/jf647/Text- > > > Indent/pull/1 to fix this, and have been in contact with the > > > current > > > maintainer regarding the patch.

> > > > Hello, > > > > I've uploaded a new version to PAUSE, 0.03, which fixes this warning > > and allows the tests to pass for installation. > > > > As soon as the index is updated and installation works again, I'll > > mark this case as resolved and close. > > > > Thanks!

> > Index is updated, confirmed the new release on CPAN, and confirmed > passing on all platforms through CPAN testers. > > Closing out.

Thanks!