Skip Menu |

This queue is for tickets about the CPAN-Mini-Inject CPAN distribution.

Report information
The Basics
Id: 84666
Status: resolved
Priority: 0/
Queue: CPAN-Mini-Inject
People
Owner: sjn [...] cpan.org
Requestors: EDENC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Apr 16 09:23:29 2013 EDENC [...] cpan.org - Ticket created
inject code copies the dist tarball once for every module, making it very slow on large releases and producing timeouts on things like CPAN::Mini::Inject::Server.
  Tue Apr 16 09:24:21 2013 EDENC [...] cpan.org - Subject changed from (no value) to 'multiple copies of tarball during inject'
  Tue Apr 16 09:59:15 2013 sjn [...] cpan.org - Correspondence added
There's a fix for this on github that has been applied but not yet released. Would you mind taking a look at it and see if it works for you?

Here's the commit:

https://github.com/wchristian/CPAN--Mini--Inject/commit/590f5a6fd17583191b6743d48ee1061f0af0f9b5

  Tue Apr 16 09:59:16 2013 The RT System itself - Status changed from 'new' to 'open'
  Tue Apr 16 09:59:16 2013 sjn [...] cpan.org - Taken
  Wed Apr 17 13:15:27 2013 EDENC [...] cpan.org - Correspondence added
On Tue Apr 16 09:59:15 2013, sjn wrote: Show quoted text
> There's a fix for this on github that has been applied but not yet > released. > Would you mind taking a look at it and see if it works for you? > > Here's the commit: > > https://github.com/wchristian/CPAN--Mini-- > Inject/commit/590f5a6fd17583191b6743d48ee1061f0af0f9b5
It should work, yes. I was trying to get a test written for it, but there's lots of details involved. Will send a pull request once it's done. Thanks
  Wed Apr 17 14:34:49 2013 MITHALDU [...] cpan.org - Correspondence added
And this one is now patched with sjn's patch and released. :)
  Wed Apr 17 14:34:50 2013 MITHALDU [...] cpan.org - Status changed from 'open' to 'resolved'