Bug #39793 for CGI-Application-Dispatch: Couldn't open t/TEST.PL

Thu Mar 27 11:02:48 2008 MARKSTOS [...] cpan.org - Ticket #34473: Ticket created

Subject:

wish: more graceful failure in old environments (2.13_02)

Trying to install this on an older FreeBSD with Apache 1.x and mod_perl 1.x failed ungracefully: perl Makefile.PL Couldn't open t/TEST.PL: No such file or directory at /usr/local/lib/perl5/site_perl/5.8.0/mach/Apache/TestMM.pm line 128. #### $ perl Build.PL Checking whether your kit is complete... Looks good Checking prerequisites... Looks good "/usr/home/mark/src/CGI-Application-Dispatch-2.13_02" is not exported by the lib module Can't continue after import errors at /usr/local/lib/perl5/site_perl/5.8.0/mach/Apache/TestConfig.pm line 223 ### I have Apache::Test 1.20. It seems like something newer is required, but upgrading past 1.21 appears not to be supported unless we also have mod_perl 2, which we don't: http://search.cpan.org/~phred/Apache-Test-1.30/Changes I don't mind if my older environment is not supported, but it would be nice to fail more gracefully, perhaps by explicitly requiring a newer version of Apache::Test. Mark

Tue Sep 16 20:28:37 2008 MARKSTOS [...] cpan.org - Ticket #34473: Correspondence added

Someone else ran into the same issue: http://www.erlbaum.net/pipermail/cgiapp/2008q1/000375.html

Tue Sep 16 20:28:39 2008 MARKSTOS [...] cpan.org - Ticket #34473: Status changed from 'new' to 'open'

Tue Sep 16 20:34:48 2008 MARKSTOS [...] cpan.org - Ticket #34473: Correspondence added

I looked at this some more today, and am not so sure it's an "old environment issue", as MP 1 is supported. It's not clear to me why it's failing.

Fri Oct 03 09:44:50 2008 kmcgrath [...] baknet.com - Ticket created

Subject:

Couldn't open t/TEST.PL

Couldn't open t/TEST.PL: No such file or directory at /usr/lib64/perl5/vendor_perl/5.8.5/x86_64-linux-thread-multi/Apache/TestMM.pm line 126. Tried to install a few different systems and always get this error. Looks like t/TEST.PL is not included in the MANIFEST which Apache::TestMM is looking for when Apache::TestMM::generate_script('./t/TEST') is called on line 9 of Makefile.PL -Kevin

Fri Oct 03 09:46:35 2008 kmcgrath [...] baknet.com - Severity Critical added

Fri Oct 17 00:42:04 2008 punkish [...] eidesis.org - Ticket #40115: Ticket created

Subject:	Couldn't open t/TEST.PL
Date:	Thu, 16 Oct 2008 23:11:13 -0500
To:	bug-CGI-Application-Dispatch [...] rt.cpan.org
From:	"Mr. Puneet Kishor" <punkish [...] eidesis.org>

- Distribution name and version: /CGI-Application-Dispatch-2.13 - Perl version: This is perl, v5.8.8 built for darwin-2level - Operating System vendor and version: Darwin Lucknow.local 9.5.0 Darwin Kernel Version 9.5.0: Wed Sep 3 11:29:43 PDT 2008; root:xnu-1228.7.58~1/RELEASE_I386 i386 i386 Unable to install. Same as ticket 39793 reported on Oct 3, 2008. CPAN.pm: Going to build M/MA/MARKSTOS/CGI-Application- Dispatch-2.13.tar.gz Couldn't open t/TEST.PL: No such file or directory at /usr/local/lib/ perl5/site_perl/5.8.8/darwin-2level/Apache/TestMM.pm line 133. Warning: No success on command[/usr/local/bin/perl Makefile.PL] MARKSTOS/CGI-Application-Dispatch-2.13.tar.gz /usr/local/bin/perl Makefile.PL -- NOT OK Running make test Make had some problems, won't test Running make install Make had some problems, won't install Failed during this command: MARKSTOS/CGI-Application-Dispatch-2.13.tar.gz: writemakefile NO '/ usr/local/bin/perl Makefile.PL' returned status 512 -- Puneet Kishor

Fri Oct 17 10:47:58 2008 mark [...] summersault.com - Ticket #40115: Correspondence added

Subject:	Re: [rt.cpan.org #40115] Couldn't open t/TEST.PL (what's the core issue?)
Date:	Fri, 17 Oct 2008 10:47:44 -0400
To:	bug-CGI-Application-Dispatch [...] rt.cpan.org
From:	Mark Stosberg <mark [...] summersault.com>

Michael Peters, Do you have any idea what causes this? I don't yet understand the root of the issue. To the reporters: Patches certainly welcome. Mark

Fri Oct 17 10:48:01 2008 The RT System itself - Ticket #40115: Status changed from 'new' to 'open'

Fri Oct 17 10:56:17 2008 wonko [...] cpan.org - Ticket #40115: Correspondence added

On Fri Oct 17 10:47:58 2008, mark@summersault.com wrote: Show quoted text

> Do you have any idea what causes this?

No clue. Show quoted text

> I don't yet understand the root of the issue.

Me either. But I've never seen this failure before. Show quoted text

> To the reporters: Patches certainly welcome.

Also, try running the install by hand to see if that changes anything. And if possible, try running Build.PL instead of Makefile.PL to see if that changes anything.

Fri Oct 17 11:04:27 2008 mark [...] summersault.com - Ticket #40115: Correspondence added

Subject:	Re: [rt.cpan.org #40115] Couldn't open t/TEST.PL
Date:	Fri, 17 Oct 2008 11:03:05 -0400
To:	bug-CGI-Application-Dispatch [...] rt.cpan.org
From:	Mark Stosberg <mark [...] summersault.com>

On Fri, 17 Oct 2008 10:56:20 -0400 "Michael Peters via RT" <bug-CGI-Application-Dispatch@rt.cpan.org> wrote: Show quoted text

> Queue: CGI-Application-Dispatch > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40115 > > > On Fri Oct 17 10:47:58 2008, mark@summersault.com wrote: >

> > Do you have any idea what causes this?

> > No clue.

Maybe we should ask upstream. It seems somehow related to an Apache testing module. Mark

Fri Oct 17 11:15:07 2008 punkish [...] eidesis.org - Ticket #40115: Correspondence added

CC:	Mark Stosberg <mark [...] summersault.com>
Subject:	Re: [rt.cpan.org #40115] Couldn't open t/TEST.PL
Date:	Fri, 17 Oct 2008 10:14:48 -0500
To:	bug-CGI-Application-Dispatch [...] rt.cpan.org
From:	"Mr. Puneet Kishor" <punkish [...] eidesis.org>

On Oct 17, 2008, at 10:04 AM, mark@summersault.com via RT wrote: Show quoted text

> <URL: http://rt.cpan.org/Ticket/Display.html?id=40115 > > > On Fri, 17 Oct 2008 10:56:20 -0400 > "Michael Peters via RT" <bug-CGI-Application-Dispatch@rt.cpan.org> > wrote: >

>> Queue: CGI-Application-Dispatch >> Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40115 > >> >> On Fri Oct 17 10:47:58 2008, mark@summersault.com wrote: >>

>>> Do you have any idea what causes this?

>> >> No clue.

> > Maybe we should ask upstream. It seems somehow related > to an Apache testing module. >

I think there is a bug in the Apache::TestMM module. It tries to create t/TEST.PL, but not in the correct place. The correct location should be the t directory in the CGI::App:D source directory. I was able to get around this by disabling testing in the Makefile.PL. I edited that file by commenting out the lines that invoke testing using Apache::TestMM. It installed just fine after that. And why wouldn't it... it is just a pure Perl module. Hope this helps in making the next update of the distribution more robust. Show quoted text

> Mark >

-- Puneet Kishor http://www.punkish.org/ Ph.D. Program, Nelson Institute, UW-Madison http://www.nelson.wisc.edu/ Charter Member, Open Source Geospatial Foundation http://www.osgeo.org/ ----------------------------------------------------------------------- collaborate, communicate, compete =======================================================================

Fri Oct 17 12:09:46 2008 MARKSTOS [...] cpan.org - Ticket #40115: Merged into ticket #39793

Fri Oct 17 12:09:46 2008 MARKSTOS [...] cpan.org - Merged into ticket #39793

Fri Oct 17 12:14:19 2008 MARKSTOS [...] cpan.org - Correspondence added

Ok, I've filed a bug with Apache::TestMM now: http://rt.cpan.org/Ticket/Display.html?id=40133 Mark

Fri Oct 17 12:14:20 2008 The RT System itself - Status changed from 'new' to 'open'

Sat Nov 01 21:38:57 2008 MARKSTOS [...] cpan.org - Taken

Sat Nov 01 21:47:32 2008 MARKSTOS [...] cpan.org - Correspondence added

RT-Send-CC:

mpeters [...] plusthree.com

Michael, What do you expect this code in Makefile.PL to do, especially the "generate_script" line? eval "require Apache::Test"; unless( $@ ) { require Apache::TestMM; Apache::TestMM->import(qw(test clean)); Apache::TestMM::filter_args(); Apache::TestMM::generate_script('t/TEST'); $have_apache_test = 1; } ### From my reading of the "Apache::TestMM" docs, it expects "t/TEST.PL" to exist, and uses this as a template to generate "t/TEST". I believe that if "t/TEST.PL" is missing, as it is in our case, then "t/TEST" will have no useful tests in it. This testing matrix shows that the related test failures seem to be specific to Linux: http://bbbike.radzeit.de/~slaven/cpantestersmatrix.cgi?dist=CGI-Application-Dispatch+2.13 The Apache::TestMM docs are not clear if "t/TEST.PL" is really optional, and it provides no example of what should go in the file. The line it is dying on looks like this: open $in, "$file.PL" or die "Couldn't open $file.PL: $!"; (where "file" would be "t/TEST"). ### I bet the reason that tests pass for most people is that they don't have Apache::Test installed, so the 'eval' files, and the "generate_script" line is never installed. Perhaps some flavors of Linux are shipping with it. So, I see two paths to resolution: 1. Quit calling "generate_script()". It doesn't appear that it was doing anything, since t/TEST.PL was missing. 2. Create and ship t/TEST.PL. Mark

Sat Nov 01 21:49:40 2008 MARKSTOS [...] cpan.org - Ticket #34473: Merged into ticket #39793

Sat Nov 01 21:49:40 2008 MARKSTOS [...] cpan.org - Merged into ticket #39793

Tue Nov 04 11:22:26 2008 MARKSTOS [...] cpan.org - Correspondence added

Subject:

Couldn't open t/TEST.PL (still with 2.14

This bug still happens with the modified Makefile.PL I shipped in 2.14. See here: http://www.nntp.perl.org/group/perl.cpan.testers/2008/11/msg2542996.html Michael, I'm considering removing the Makefile.PL, since the Build.PL seems not to have a problem. I suspect most modern CPAN install toolchains will be able to cope with this. Any objections? I'm not particularly interested in getting into the guts of Apache::Test to solve this another way, when we already have one working solution for this. Mark

Mon Mar 23 21:56:17 2009 MARKSTOS [...] cpan.org - Correspondence added

I'm going to put out a new release which removes Makefile.PL. If it happens again after the 2.16 release, re-open. (Modern toolchains should pick up Build.PL instead).

Mon Mar 23 21:56:18 2009 MARKSTOS [...] cpan.org - Status changed from 'open' to 'resolved'