Bug #60410 for local-lib: failing tests

Mon Aug 16 20:33:29 2010 xenoterracide [...] gmail.com - Ticket created

CC:	Justin Davis <jrcd83 [...] gmail.com>
Subject:	failing tests
Date:	Mon, 16 Aug 2010 20:33:19 -0400
To:	bugs-local-lib [...] rt.cpan.org
From:	Caleb Cushing <xenoterracide [...] gmail.com>

local::lib seems to be failing most of its tests... could be a problem with CPANPLUS::Dist::Arch so cc-ing its author. But even if it is a specific interaction with that, it's also possible the tests could be written in a way that they would still work. Skip blib/lib/POD2/PT_BR/local/lib.pod (unchanged) Skip blib/lib/lib/core/only.pm (unchanged) Skip blib/lib/local/lib.pm (unchanged) Skip blib/lib/POD2/DE/local/lib.pod (unchanged) Manifying blib/man3/POD2::PT_BR::local::lib.3pm Manifying blib/man3/lib::core::only.3pm Manifying blib/man3/local::lib.3pm Manifying blib/man3/POD2::DE::local::lib.3pm PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'inc', 'blib/lib', 'blib/arch')" t/classmethod.t t/coderefs_in_inc.t t/de-dup.t t/install.t t/lib-core-only.t t/pipeline.t t/classmethod.t ...... Name "File::Spec::rel2abs" used only once: possible typo at t/classmethod.t line 20. t/classmethod.t ...... 1/? Attempting to create directory t/var/splat Attempting to create file t/var/splat/.modulebuildrc t/classmethod.t ...... ok t/coderefs_in_inc.t .. Attempting to create file /home/xenoterracide/.cpanplus/5.12.1/pacman/build/perl-local-lib-1.006007/src/local-lib-1.006007/t/test_local_lib-g7ZYa/.modulebuildrc t/coderefs_in_inc.t .. ok t/de-dup.t ........... Attempting to create file /home/xenoterracide/.cpanplus/5.12.1/pacman/build/perl-local-lib-1.006007/src/local-lib-1.006007/t/test_local_lib-9hyhq/.modulebuildrc t/de-dup.t ........... ok t/install.t .......... Attempting to create file /home/xenoterracide/.cpanplus/5.12.1/pacman/build/perl-local-lib-1.006007/src/local-lib-1.006007/t/test_local_lib-u8oG6/.modulebuildrc t/install.t .......... 1/2 # Failed test 'MB.pm installed into the correct location' # at t/install.t line 35. # Looks like you failed 1 test of 2. t/install.t .......... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/2 subtests t/lib-core-only.t .... ok t/pipeline.t ......... ok Test Summary Report ------------------- t/install.t (Wstat: 256 Tests: 2 Failed: 1) Failed test: 2 Non-zero exit status: 1 Files=6, Tests=13, 1 wallclock secs ( 0.03 usr 0.01 sys + 0.77 cusr 0.13 csys = 0.94 CPU) Result: FAIL Failed 1/6 test programs. 1/13 subtests failed. make: *** [test_dynamic] Error 255 Aborting... [ERROR] makepkg returned abnormal status: 2 [ERROR] Unable to create a new distribution object for 'local::lib' -- cannot continue -- Caleb Cushing http://xenoterracide.com

Message body is not shown because sender requested not to inline it.

Wed Aug 18 21:37:39 2010 JUSTER [...] cpan.org - Correspondence added

I am the author of CPANPLUS::Dist::Arch, a backend for CPANPLUS that builds CPAN modules as packages for ArchLinux. My module uses environment variables to pass parameters to Module::Build and ExtUtils::MakeMaker/Module::Install. Ironically I learned about the environment vars from this module :). All of the env. variables my module uses are overriden by local::lib except for PERL_MB_OPT, which I think is causing the problem. This is also a possible problem in other locations, where users have customized their environment with this same environment variable. It is similar to PERL_MM_OPT but takes Module::Build specific parameters. I would suggest adding the following line, before the for loop, to unset the problem-causing environment variable just in case a user has it set or some automatic packager like mine is using it: local $ENV{PERL_MB_OPT}; On Mon Aug 16 20:33:29 2010, XENO wrote: Show quoted text

> t/install.t .......... Attempting to create file > /home/xenoterracide/.cpanplus/5.12.1/pacman/build/perl-local-lib- > 1.006007/src/local-lib-1.006007/t/test_local_lib-u8oG6/.modulebuildrc > t/install.t .......... 1/2 > # Failed test 'MB.pm installed into the correct location' > # at t/install.t line 35. > # Looks like you failed 1 test of 2.

Wed Aug 18 21:37:40 2010 The RT System itself - Status changed from 'new' to 'open'

Wed Aug 18 21:40:47 2010 GETTY [...] cpan.org - Correspondence added

RT-Send-CC:

jrcd83 [...] gmail.com, bugs-local-lib [...] rt.cpan.org

Could you join #local-lib on irc.perl.org so that we can discuss this through?

Wed Aug 18 22:41:09 2010 jrcd83 [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #60410] failing tests
Date:	Wed, 18 Aug 2010 19:40:56 -0700
To:	bug-local-lib [...] rt.cpan.org
From:	Justin Davis <jrcd83 [...] gmail.com>

Here is the patch generated from git format-patch. You should be able to apply it by typing 'git apply <patch filename>'. From ea6d110a4bb2e0e33bc4ab527549b778e963ac5b Mon Sep 17 00:00:00 2001 From: Justin Davis <jrcd83@gmail.com> Date: Wed, 18 Aug 2010 19:29:56 -0700 Subject: [PATCH] Unset PERL_MB_OPT env. var in case it sets --destdir. In such a case the install.t will fail erronously. --- t/install.t | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/t/install.t b/t/install.t index 67cfa17..3e8edec 100644 --- a/t/install.t +++ b/t/install.t @@ -16,6 +16,9 @@ my $dir = tempdir('test_local_lib-XXXXX', DIR => Cwd::abs_path('t'), CLEANUP => use local::lib (); local::lib->import($dir); +# Unset PERL_MB_OPT in case it uses --destdir +local $ENV{ 'PERL_MB_OPT' }; + my $orig_dir = cwd; SKIP: for my $dist_type (qw(EUMM MB)) { chdir File::Spec->catdir($orig_dir, qw(t dist), $dist_type); -- 1.7.1 Thanks, Justin On Aug 18, 2010, at 6:40 PM, Torsten Raudssus via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=60410 > > > Could you join #local-lib on irc.perl.org so that we can discuss this > through?

Wed Aug 18 22:43:55 2010 rafl [...] debian.org - Correspondence added

Subject:	Re: [rt.cpan.org #60410] failing tests
Date:	Thu, 19 Aug 2010 04:42:54 +0200
To:	bug-local-lib [...] rt.cpan.org
From:	Florian Ragwitz <rafl [...] debian.org>

"Justin Davis via RT" <bug-local-lib@rt.cpan.org> writes: Show quoted text

> Here is the patch generated from git format-patch. You should be able > to apply it by typing 'git apply <patch filename>'.

ITYM: git am $patch

Download (untitled)
application/pgp-signature 197b

Message body not shown because it is not plain text.

Wed Jul 13 22:28:53 2011 apeiron [...] cpan.org - Correspondence added

toolchain error on part of reporter

Wed Jul 13 22:28:53 2011 apeiron [...] cpan.org - Status changed from 'open' to 'rejected'