Bug #98272 for podlators: Test failures in termcap tests

Fri Aug 22 19:51:37 2014 kalaria [...] gmail.com - Ticket created

Subject:	Install Error
Date:	Fri, 22 Aug 2014 19:51:10 -0400
To:	bug-podlators [...] rt.cpan.org
From:	Janak <kalaria [...] gmail.com>

Show quoted text

> > --> Working on Pod::Man > > Fetching http://www.cpan.org/authors/id/R/RR/RRA/podlators-2.5.3.tar.gz

>> ... OK

> > Configuring podlators-v2.5.3 ... OK > > Building and testing podlators-v2.5.3 ... FAIL > > ! Installing Pod::Man failed. See

>> /home/jadaka/.cpanm/work/1408751013.7377/build.log for details. Retry with >> --force to force install it.

> > jadaka@arjun:~$ cat /home/jadaka/.cpanm/work/1408751013.7377/build.log > > cpanm (App::cpanminus) 1.7004 on perl 5.018002 built for

>> x86_64-linux-gnu-thread-multi

> > Work directory is /home/jadaka/.cpanm/work/1408751013.7377 > > You have make /usr/bin/make > > You have LWP 6.08 > > You have /bin/tar: tar (GNU tar) 1.27.1 > > Copyright (C) 2013 Free Software Foundation, Inc. > > License GPLv3+: GNU GPL version 3 or later <

>> http://gnu.org/licenses/gpl.html>.

> > This is free software: you are free to change and redistribute it. > > There is NO WARRANTY, to the extent permitted by law. > >

>> Written by John Gilmore and Jay Fenlason.

> > You have /usr/bin/unzip > > Searching Pod::Man on cpanmetadb ... > > --> Working on Pod::Man > > Fetching http://www.cpan.org/authors/id/R/RR/RRA/podlators-2.5.3.tar.gz > > -> OK > > Unpacking podlators-2.5.3.tar.gz > > Entering podlators-2.5.3 > > Checking configure dependencies from META.json > > Checking if you have ExtUtils::MakeMaker 0 ... Yes (6.98) > > Configuring podlators-v2.5.3 > > Running Makefile.PL > > Checking if your kit is complete... > > Looks good > > Generating a Unix-style Makefile > > Writing Makefile for Pod > > Writing MYMETA.yml and MYMETA.json > > -> OK > > Checking dependencies from MYMETA.json ... > > Checking if you have Pod::Simple 3.06 ... Yes (3.28) > > Checking if you have File::Spec 0.8 ... Yes (3.47) > > Checking if you have Encode 0 ... Yes (2.62) > > Checking if you have ExtUtils::MakeMaker 0 ... Yes (6.98) > > Building and testing podlators-v2.5.3 > > cp lib/Pod/Text/Termcap.pm blib/lib/Pod/Text/Termcap.pm > > cp lib/Pod/Text/Color.pm blib/lib/Pod/Text/Color.pm > > cp lib/Pod/Text.pm blib/lib/Pod/Text.pm > > cp lib/Pod/ParseLink.pm blib/lib/Pod/ParseLink.pm > > cp lib/Pod/Text/Overstrike.pm blib/lib/Pod/Text/Overstrike.pm > > cp lib/Pod/Man.pm blib/lib/Pod/Man.pm > > cp scripts/pod2man blib/script/pod2man > > /usr/bin/perl -MExtUtils::MY -e 'MY->fixin(shift)' -- blib/script/pod2man > > cp scripts/pod2text blib/script/pod2text > > /usr/bin/perl -MExtUtils::MY -e 'MY->fixin(shift)' -- blib/script/pod2text > > Manifying blib/man1/perlpodstyle.1p > > Manifying blib/man1/pod2man.1p > > Manifying blib/man1/pod2text.1p > > Manifying blib/man3/Pod::Man.3pm > > Manifying blib/man3/Pod::ParseLink.3pm > > Manifying blib/man3/Pod::Text.3pm > > Manifying blib/man3/Pod::Text::Color.3pm > > Manifying blib/man3/Pod::Text::Overstrike.3pm > > Manifying blib/man3/Pod::Text::Termcap.3pm > > PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM"

>> "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, >> 'blib/lib', 'blib/arch')" t/*.t

> >

>> # Failed test 'Pod::Text::Termcap output is correct'

> > # at t/basic.t line 111. > > # Non-matching output left in out7450.cap > > # Looks like you failed 1 test of 15. > > t/basic.t .......... > > Dubious, test returned 1 (wstat 256, 0x100) > > Failed 1/15 subtests > > t/color.t .......... ok > > t/devise-date.t .... ok > > t/filehandle.t ..... ok > > t/man-empty.t ...... ok > > t/man-heading.t .... ok > > t/man-options.t .... ok > > t/man-perlio.t ..... ok > > t/man-utf8.t ....... ok > > t/man.t ............ ok > > t/overstrike.t ..... ok > > t/parselink.t ...... ok > > t/pod-parser.t ..... ok > > t/pod-spelling.t ... skipped: Spelling tests only run for maintainer > > t/pod.t ............ ok > >

>> # Failed test 'Output correct for test 1'

> > # at t/termcap.t line 57. > > # got: 'WRAPPING > > # Do not include formatting codes when wrapping. > > # > > # ' > > # expected: 'WRAPPING > > # Do not include formatting codes when wrapping. > > # > > # ' > >

>> # Failed test 'Output correct for test 2'

> > # at t/termcap.t line 57. > > # got: 'TAG WIDTH > > # 12345678 A > > # > > # 12345678 B > > # > > # 1 C > > # > > # 1 D > > # > > # ' > > # expected: 'TAG WIDTH > > # 12345678 A > > # > > # 12345678 B > > # > > # 1 C > > # > > # 1 D > > # > > # ' > > # Looks like you failed 2 tests of 4. > > t/termcap.t ........ > > Dubious, test returned 2 (wstat 512, 0x200) > > Failed 2/4 subtests > > t/text-empty.t ..... ok > > t/text-encoding.t .. ok > > t/text-options.t ... ok > > t/text-perlio.t .... ok > > t/text-utf8.t ...... ok > > t/text.t ........... ok > >

>> Test Summary Report

> > ------------------- > > t/basic.t (Wstat: 256 Tests: 15 Failed: 1) > > Failed test: 15 > > Non-zero exit status: 1 > > t/termcap.t (Wstat: 512 Tests: 4 Failed: 2) > > Failed tests: 3-4 > > Non-zero exit status: 2 > > Files=22, Tests=232, 2 wallclock secs ( 0.07 usr 0.01 sys + 1.05 cusr

>> 0.10 csys = 1.23 CPU)

> > Result: FAIL > > Failed 2/22 test programs. 3/232 subtests failed. > > make: *** [test_dynamic] Error 255 > >

Message body is not shown because it is too large.

Sat Aug 23 01:31:24 2014 RRA [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #98272] Install Error
Date:	Fri, 22 Aug 2014 22:31:13 -0700
To:	bug-podlators [...] rt.cpan.org
From:	Russ Allbery <rra [...] cpan.org>

"Janak via RT" <bug-podlators@rt.cpan.org> writes: Show quoted text

>>> # Failed test 'Pod::Text::Termcap output is correct'

>> # at t/basic.t line 111. >> # Non-matching output left in out7450.cap >> # Looks like you failed 1 test of 15. >> t/basic.t .......... >> Dubious, test returned 1 (wstat 256, 0x100) >> Failed 1/15 subtests

Yeah, Term::Cap broke compatibility with how the test suite is set up. It only affects the test suite, not the module. I keep hoping the bug will be fixed in Term::Cap, mostly because I'm insanely busy at the moment, but it's on my to-do list if I can ever find some free time for a new release. -- #!/usr/bin/perl -- Russ Allbery, Just Another Perl Hacker $^=q;@!>~|{>krw>yn{u<$$<[~||<Juukn{=,<S~|}<Jwx}qn{<Yn{u<Qjltn{ > 0gFzD gD, 00Fz, 0,,( 0hF 0g)F/=, 0> "L$/GEIFewe{,$/ 0C$~> "@=,m,|,(e 0.), 01,pnn,y{ rw} >;,$0=q,$,,($_=$^)=~y,$/ C-~><@=\n\r,-~$:-u/ #y,d,s,(\$.),$1,gee,print

Sat Aug 23 01:31:24 2014 The RT System itself - Status changed from 'new' to 'open'

Sat Aug 30 18:29:57 2014 RRA [...] cpan.org - Subject changed from 'Install Error' to 'Test failures in termcap tests'

Thu Sep 04 06:02:28 2014 SREZIC [...] cpan.org - Dependency on ticket #98608 added

Wed Nov 12 06:15:39 2014 SREZIC [...] cpan.org - Correspondence added

On 2014-08-23 01:31:24, RRA wrote: Show quoted text

> "Janak via RT" <bug-podlators@rt.cpan.org> writes: >

> >>> # Failed test 'Pod::Text::Termcap output is correct'

> >> # at t/basic.t line 111. > >> # Non-matching output left in out7450.cap > >> # Looks like you failed 1 test of 15. > >> t/basic.t .......... > >> Dubious, test returned 1 (wstat 256, 0x100) > >> Failed 1/15 subtests

> > Yeah, Term::Cap broke compatibility with how the test suite is set up. > It only affects the test suite, not the module. I keep hoping the bug > will be fixed in Term::Cap, mostly because I'm insanely busy at the > moment, but it's on my to-do list if I can ever find some free time for a > new release.

I looked at the Term::Cap 1.16 change, and it looks reasonable to me. Maybe it's better to fix the podlators test suite, see the attached perl. Should hopefully work with old and new Term::Cap.

Subject:

0001-fix-for-RT-98272.patch

From 6cd1694c72ea7ae4fa4d5c247a7b968fe12d09cb Mon Sep 17 00:00:00 2001 From: Slaven Rezic <slaven.rezic@idealo.de> Date: Wed, 12 Nov 2014 12:12:03 +0100 Subject: [PATCH] fix for RT #98272 --- t/basic.t | 3 ++- t/termcap.t | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/t/basic.t b/t/basic.t index 4103ed6..5756107 100755 --- a/t/basic.t +++ b/t/basic.t @@ -44,7 +44,8 @@ sub source_path { # Hard-code a few values to try to get reproducible results. $ENV{COLUMNS} = 80; -$ENV{TERM} = 'xterm'; +#$ENV{TERM} = 'xterm'; +delete $ENV{TERM}; $ENV{TERMCAP} = 'xterm:co=80:do=^J:md=\E[1m:us=\E[4m:me=\E[m'; # Map of translators to file extensions to find the formatted output to diff --git a/t/termcap.t b/t/termcap.t index d751bad..ac919aa 100755 --- a/t/termcap.t +++ b/t/termcap.t @@ -24,7 +24,8 @@ BEGIN { use_ok ('Pod::Text::Termcap') } # Hard-code a few values to try to get reproducible results. $ENV{COLUMNS} = 80; -$ENV{TERM} = 'xterm'; +#$ENV{TERM} = 'xterm'; +delete $ENV{TERM}; $ENV{TERMCAP} = 'xterm:co=80:do=^J:md=\E[1m:us=\E[4m:me=\E[m'; my $parser = Pod::Text::Termcap->new; -- 1.7.9.5

Mon Apr 20 10:17:10 2015 SREZIC [...] cpan.org - Correspondence added

On 2014-11-12 06:15:39, SREZIC wrote: Show quoted text

> On 2014-08-23 01:31:24, RRA wrote:

> > "Janak via RT" <bug-podlators@rt.cpan.org> writes: > >

> > >>> # Failed test 'Pod::Text::Termcap output is correct'

> > >> # at t/basic.t line 111. > > >> # Non-matching output left in out7450.cap > > >> # Looks like you failed 1 test of 15. > > >> t/basic.t .......... > > >> Dubious, test returned 1 (wstat 256, 0x100) > > >> Failed 1/15 subtests

> > > > Yeah, Term::Cap broke compatibility with how the test suite is set > > up. > > It only affects the test suite, not the module. I keep hoping the > > bug > > will be fixed in Term::Cap, mostly because I'm insanely busy at the > > moment, but it's on my to-do list if I can ever find some free time > > for a > > new release.

> > I looked at the Term::Cap 1.16 change, and it looks reasonable to me. > Maybe it's better to fix the podlators test suite, see the attached > perl. Should hopefully work with old and new Term::Cap.

Any progress here? Is it a bug in Term::Cap or in podlators? Could the proposed patch be applied?

Mon Apr 20 22:22:17 2015 RRA [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #98272] Test failures in termcap tests
Date:	Mon, 20 Apr 2015 19:22:00 -0700
To:	"Slaven_Rezic via RT" <bug-podlators [...] rt.cpan.org>
From:	Russ Allbery <rra [...] cpan.org>

"Slaven_Rezic via RT" <bug-podlators@rt.cpan.org> writes: Show quoted text

> Any progress here? Is it a bug in Term::Cap or in podlators? Could the > proposed patch be applied?

podlators is fixed in Git. I have another day or two of work required to get a release out. I got a chunk of work finished on this last week, and might be able to finish this week. -- #!/usr/bin/perl -- Russ Allbery, Just Another Perl Hacker $^=q;@!>~|{>krw>yn{u<$$<[~||<Juukn{=,<S~|}<Jwx}qn{<Yn{u<Qjltn{ > 0gFzD gD, 00Fz, 0,,( 0hF 0g)F/=, 0> "L$/GEIFewe{,$/ 0C$~> "@=,m,|,(e 0.), 01,pnn,y{ rw} >;,$0=q,$,,($_=$^)=~y,$/ C-~><@=\n\r,-~$:-u/ #y,d,s,(\$.),$1,gee,print

Sun Nov 29 04:38:35 2015 SREZIC [...] cpan.org - Correspondence added

On 2015-04-20 22:22:17, RRA wrote: Show quoted text

> "Slaven_Rezic via RT" <bug-podlators@rt.cpan.org> writes: >

> > Any progress here? Is it a bug in Term::Cap or in podlators? Could the > > proposed patch be applied?

> > podlators is fixed in Git. I have another day or two of work required to > get a release out. I got a chunk of work finished on this last week, and > might be able to finish this week. >

podlators-4.00 still fails if Term::Cap 1.16 is installed --- which is responsible for the red spots in the CPAN Testers matrix: http://fast-matrix.cpantesters.org/?dist=podlators%204.00

Sun Dec 06 18:14:23 2015 RRA [...] cpan.org - Fixed in 4.02 added

Sun Dec 06 18:14:23 2015 RRA [...] cpan.org - Correspondence added

This should now really be fixed. Verified with a local installation of Term::Cap and podlators 4.02. Sorry about the ridiculous delay on my part in getting this cleared up.

Sun Dec 06 18:14:25 2015 RRA [...] cpan.org - Taken

Sun Dec 06 18:14:56 2015 RRA [...] cpan.org - Dependency on ticket #98608 deleted

Sun Dec 06 18:15:08 2015 RRA [...] cpan.org - Status changed from 'open' to 'resolved'