Skip Menu |
 

This queue is for tickets about the ExtUtils-Install CPAN distribution.

Report information
The Basics
Id: 40700
Status: open
Priority: 0/
Queue: ExtUtils-Install

People
Owner: Nobody in particular
Requestors: craigberry [...] mac.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: test file nit in 1.50_04
Date: Wed, 05 Nov 2008 19:33:05 -0600
To: bug-ExtUtils-Install [...] rt.cpan.org
From: "Craig A. Berry" <craigberry [...] mac.com>
Download (untitled) / with headers
text/plain 913b
On Nov 5, 2008, at 3:13 PM, Yves via RT wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=34095 > > > I think this is resolved by applying the changes from the bleadperl > release. Please advise if it is not fixed in 1.50_04
Everything passes after the tweak below. This was not platform- specific. Before the change, basic.t failed on both Mac OS X and VMS because it was trying to run basic.plt rather than basic.pl. On VMS I tested with a recent blead as well as maint-5.8@34718. --- t/basic.t;-0 Wed Nov 5 14:49:08 2008 +++ t/basic.t Wed Nov 5 19:15:10 2008 @@ -28,3 +28,3 @@ $::TEST_INSTALL_ONLY= 1; -(my $file=$0)=~s/\.t$/.plt/; +(my $file=$0)=~s/\.t$/.pl/; do $file; [end of patch] Show quoted text
________________________________________ Craig A. Berry mailto:craigberry@mac.com "... getting out of a sonnet is much more difficult than getting in." Brad Leithauser
CC: craigberry [...] mac.com
Subject: Re: [rt.cpan.org #40700] test file nit in 1.50_04
Date: Thu, 6 Nov 2008 10:32:40 +0100
To: bug-ExtUtils-Install [...] rt.cpan.org
From: demerphq <demerphq [...] gmail.com>
Download (untitled) / with headers
text/plain 1.4k
2008/11/6 craigberry@mac.com via RT <bug-ExtUtils-Install@rt.cpan.org>: Show quoted text
> Wed Nov 05 20:33:25 2008: Request 40700 was acted upon. > Transaction: Ticket created by craigberry@mac.com > Queue: ExtUtils-Install > Subject: test file nit in 1.50_04 > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: craigberry@mac.com > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40700 > > > > > On Nov 5, 2008, at 3:13 PM, Yves via RT wrote: >
>> <URL: http://rt.cpan.org/Ticket/Display.html?id=34095 > >> >> I think this is resolved by applying the changes from the bleadperl >> release. Please advise if it is not fixed in 1.50_04
> > Everything passes after the tweak below. This was not platform- > specific. Before the change, basic.t failed on both Mac OS X and VMS > because it was trying to run basic.plt rather than basic.pl. On VMS I > tested with a recent blead as well as maint-5.8@34718. > > --- t/basic.t;-0 Wed Nov 5 14:49:08 2008 > +++ t/basic.t Wed Nov 5 19:15:10 2008 > @@ -28,3 +28,3 @@ $::TEST_INSTALL_ONLY= 1; > > -(my $file=$0)=~s/\.t$/.plt/; > +(my $file=$0)=~s/\.t$/.pl/; > do $file; > [end of patch]
Gah, stupid me. I renamed things but because I had an old version in the t directory with the old name my tests passed even when they shouldn't have. Thanks, applied, and a 1.50_05 released. Ill give it a day or two before i rename it 1.51 Cheers, yves -- perl -Mre=debug -e "/just|another|perl|hacker/"
Download (untitled) / with headers
text/plain 1.7k
On Thu Nov 06 04:32:51 2008, demerphq@gmail.com wrote: Show quoted text
> 2008/11/6 craigberry@mac.com via RT <bug-ExtUtils-Install@rt.cpan.org>:
> > Wed Nov 05 20:33:25 2008: Request 40700 was acted upon. > > Transaction: Ticket created by craigberry@mac.com > > Queue: ExtUtils-Install > > Subject: test file nit in 1.50_04 > > Broken in: (no value) > > Severity: (no value) > > Owner: Nobody > > Requestors: craigberry@mac.com > > Status: new > > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40700 > > > > > > > > > On Nov 5, 2008, at 3:13 PM, Yves via RT wrote: > >
> >> <URL: http://rt.cpan.org/Ticket/Display.html?id=34095 > > >> > >> I think this is resolved by applying the changes from the bleadperl > >> release. Please advise if it is not fixed in 1.50_04
> > > > Everything passes after the tweak below. This was not platform- > > specific. Before the change, basic.t failed on both Mac OS X and VMS > > because it was trying to run basic.plt rather than basic.pl. On VMS I > > tested with a recent blead as well as maint-5.8@34718. > > > > --- t/basic.t;-0 Wed Nov 5 14:49:08 2008 > > +++ t/basic.t Wed Nov 5 19:15:10 2008 > > @@ -28,3 +28,3 @@ $::TEST_INSTALL_ONLY= 1; > > > > -(my $file=$0)=~s/\.t$/.plt/; > > +(my $file=$0)=~s/\.t$/.pl/; > > do $file; > > [end of patch]
> > Gah, stupid me. I renamed things but because I had an old version in > the t directory with the old name my tests passed even when they > shouldn't have. > > Thanks, applied, and a 1.50_05 released. > > Ill give it a day or two before i rename it 1.51 > > Cheers, > yves > > >
This ticket looks closable. There is no longer a t/basic.t file in the ExtUtils-Install CPAN distribution (as of v2.04). Thank you very much. Jim Keenan


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.