Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Email-Send CPAN distribution.

Report information
The Basics
Id: 16677
Status: resolved
Priority: 0/
Queue: Email-Send

People
Owner: Nobody in particular
Requestors: cpan [...] ali.as
Cc:
AdminCc:

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



Subject: Cannot install Email::Send 2.00 when old Email::Send::Test is around
Email::Send does a can_ok check on every driver installed on the system. So failures in code outside of Email::Send can cause Email::Send's tests to fail spuriously. For example, Email::Send::Test was written again Email::Send 1.xx, which did not have an is_available method. So if an old Email::Send::Test is installed, tests in Email::Send 2.00 fail, and you can't upgrade Email::Send. Bad. Recommendations: 1. Only check the can_ok for drivers from the core distribution. 2. Suck Email::Send::Test into the main distribution. It's only 10-20 lines of code and it solves the problem. They really belong together anyway.
This has been causing upgraders to have their CPAN installers get stuck in a circular dep and fail for 5 weeks now. I've fixed this by unilaterally merging Email-Send with Email-Send-Test. I've merged the libs, and moved in Email::Send::Test's tests too. They belong together anyway. I've fixed all the relevant PAUSE permissions to make this happen, I've uploaded Email::Send 2.02 myself as an emergency fix. You should be able to do future uploads as normal, with the combined dist.
Subject: Re: [rt.cpan.org #16677] Cannot install Email::Send 2.00 when old Email::Send::Test is around
Date: Sat, 28 Jan 2006 13:46:19 -0800
To: bug-Email-Send [...] rt.cpan.org
From: Casey West <casey [...] geeknest.com>
On Jan 28, 2006, at 9:10 AM, via RT wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=16677 > > > This has been causing upgraders to have their CPAN installers get > stuck > in a circular dep and fail for 5 weeks now. >
I uploaded Email Send version 2.01 on the 17th of January. It has a small mistake which still caused tests to fail. I've uploaded Email::Send version 2.03 which fixes this problem. Thanks for the report. Show quoted text
> I've fixed this by unilaterally merging Email-Send with Email-Send- > Test. > > I've merged the libs, and moved in Email::Send::Test's tests too. They > belong together anyway. > > I've fixed all the relevant PAUSE permissions to make this happen, > I've > uploaded Email::Send 2.02 myself as an emergency fix. > > You should be able to do future uploads as normal, with the > combined dist.
Please submit a patch to me for this feature addition. Email::Send version 2.03 does not contain these modifications. Thanks. -- Casey West
Subject: Re: [rt.cpan.org #16677] Cannot install Email::Send 2.00 when old Email::Send::Test is around
Date: Sun, 29 Jan 2006 09:05:09 +1100
To: bug-Email-Send [...] rt.cpan.org
From: Adam Kennedy <adam [...] phase-n.com>
There was no patch required. Take the three files from Email-Send-Test 2.00... lib/Email/Send/Test.pm t/01_compile.t t/02_main.t Place them in the Email-Send tarball. Done Casey West via RT wrote: Show quoted text
> On Jan 28, 2006, at 9:10 AM, via RT wrote: >
>> <URL: http://rt.cpan.org/Ticket/Display.html?id=16677 > >> >> This has been causing upgraders to have their CPAN installers get >> stuck >> in a circular dep and fail for 5 weeks now. >>
> > I uploaded Email Send version 2.01 on the 17th of January. It has a > small mistake which still caused tests to fail. I've uploaded > Email::Send version 2.03 which fixes this problem. Thanks for the > report. >
>> I've fixed this by unilaterally merging Email-Send with Email-Send- >> Test. >> >> I've merged the libs, and moved in Email::Send::Test's tests too. They >> belong together anyway. >> >> I've fixed all the relevant PAUSE permissions to make this happen, >> I've >> uploaded Email::Send 2.02 myself as an emergency fix. >> >> You should be able to do future uploads as normal, with the >> combined dist.
> > Please submit a patch to me for this feature addition. Email::Send > version 2.03 does not contain these modifications. > > Thanks. >