exodist / Term-Table

Format a header and rows into a table
Other
6 stars 9 forks source link

build issues on clean 5.26.3 installation #24

Open djerius opened 4 weeks ago

djerius commented 4 weeks ago

Installation of Term::Table 0.022 fails on a clean 5.26.3 because (I think) it requires a newer version of Test2::Tools::Tiny.

Here's a minimally edited log (using plenv to create the clean install)

% plenv install --as 5.26-base -j 4 5.26.3
[...]

% plenv shell 5.26-base
% plenv which perl
/home/dj/.plenv/versions/5.26-base/bin/perl

% perl -v

This is perl 5, version 26, subversion 3 (v5.26.3) built for x86_64-linux
(with 1 registered patch, see perl -V for more detail)
[...]

% perldoc -l Test2::Tools::Tiny
/home/dj/.plenv/versions/5.26-base/lib/perl5/5.26.3/Test2/Tools/Tiny.pm

% perl -MTest2::Tools::Tiny -E 'say Test2::Tools::Tiny->VERSION'
1.302073

% wget -q https://cpan.metacpan.org/authors/id/E/EX/EXODIST/Term-Table-0.022.tar.gz
% tar xf Term-Table-0.022.tar.gz 
% cd Term-Table-0.022/
% perl Makefile.PL
[...]

% make test

PERL_DL_NONLAZY=1 "/home/dj/.plenv/versions/5.26-base/bin/perl5.26.3" "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/Table/*.t
t/bad_blank_line.t ........ ok   
t/HashBase.t .............. ok    
t/honor_env_in_non_tty.t .. No subtests run 
t/issue-9.t ............... ok    
t/Table.t ................. ok   
t/Table/Cell.t ............ ok   
t/Table/CellStack.t ....... ok   
t/Table/LineBreak.t ....... ok   

Test Summary Report
-------------------
t/honor_env_in_non_tty.t (Wstat: 0 Tests: 0 Failed: 0)
  Parse errors: No plan found in TAP output
Files=8, Tests=77,  0 wallclock secs ( 0.02 usr  0.02 sys +  0.23 cusr  0.04 csys =  0.31 CPU)
Result: FAIL
Failed 1/8 test programs. 0/77 subtests failed.
make: *** [Makefile:877: test_dynamic] Error 255
% 

% perl -Ilib t/honor_env_in_non_tty.t
# Using Test2::Tools::Tiny

No TAP output.

Try installing the latest Test2::Tools::Tiny:

The first attempt fails because Test::Simple depends on Term::Table, which fails to install (same error as above, of course)

% perl ~/bin/cpanm -l local  Test2::Tools::Tiny
--> Working on Test2::Tools::Tiny
Fetching http://www.cpan.org/authors/id/E/EX/EXODIST/Test-Simple-1.302204.tar.gz ... OK
Configuring Test-Simple-1.302204 ... OK
==> Found dependencies: Term::Table
--> Working on Term::Table
Fetching http://www.cpan.org/authors/id/E/EX/EXODIST/Term-Table-0.022.tar.gz ... OK
Configuring Term-Table-0.022 ... OK
Building and testing Term-Table-0.022 ... FAIL
! Installing Term::Table failed. See /home/dj/.cpanm/work/1729889075.1257958/build.log for details. Retry with --force to force install it.
! Installing the dependencies failed: Module 'Term::Table' is not installed
! Bailing out the installation for Test-Simple-1.302204.
% 

Force install:

% perl ~/bin/cpanm -l local -f Test2::Tools::Tiny
--> Working on Test2::Tools::Tiny
! Failed to download file:///home/dj/Work/darkpan/authors/id/E/EX/EXODIST/Test-Simple-1.302204.tar.gz
Fetching http://www.cpan.org/authors/id/E/EX/EXODIST/Test-Simple-1.302204.tar.gz ... OK
Configuring Test-Simple-1.302204 ... OK
==> Found dependencies: Term::Table
--> Working on Term::Table
Fetching http://www.cpan.org/authors/id/E/EX/EXODIST/Term-Table-0.022.tar.gz ... OK
Configuring Term-Table-0.022 ... OK
Building and testing Term-Table-0.022 ... FAIL
! Testing Term-Table-0.022 failed but installing it anyway.
Successfully installed Term-Table-0.022
Building and testing Test-Simple-1.302204 ... 
Successfully installed Test-Simple-1.302204 (upgraded from 1.302073)
2 distributions installed

Rerun the tests using the new Test2::Tools::Tiny without the new Term::Table (doesn't make a difference, but just in case)

% rm -rf local/lib/perl5/Term/Table.pm
% perl -Ilib -Ilocal/lib/perl5 t/honor_env_in_non_tty.t
# Using Test2::Tools::Tiny
ok 1 - STDOUT is not a term
ok 2 - Not using Term::Readkey without a term
ok 3 - Not using Term::Size::Any without a term
ok 4 - Get default size without the var
ok 5 - Used the size in the env var
1..5
exodist commented 4 weeks ago

Thanks for the report. I will fix it when I have a moment, or will gladly merge a PR to fix it if you beat me to it.

djerius commented 3 weeks ago

I've come up with a patch; am in the midst of putting it together.

djerius commented 3 weeks ago

See #25