Skip Menu |
 

This queue is for tickets about the Array-Compare CPAN distribution.

Report information
The Basics
Id: 49270
Status: rejected
Priority: 0/
Queue: Array-Compare

People
Owner: DAVECROSS [...] cpan.org
Requestors: adamk [...] cpan.org
Cc:
AdminCc:

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



Subject: Remove the use of Moose
Download (untitled) / with headers
text/plain 404b
Array::Compare has switched to Moose. This adds a huge amount of additional dependencies, and the slower process startup time means Array::Compare is not longer usable in any command line applications that need to be snappy (or anything else that cares a lot about startup time, like Padre). If this can't be fixed, we're going to need to remove Array::Compare (and everything that uses it) from Padre.
Download (untitled) / with headers
text/plain 277b
Correction. Test::Warn has ditched Array::Compare for us (Although I think they were going to do it anyway even before the Moose switch) so this now only has an impact of the 4 or 5 remaining modules that use Array::Compare, non of which look useful in a command line context.
Download (untitled) / with headers
text/plain 295b
I agree. Something with such a simple fixed scope shouldn't use Moose. If a user wants to use it with a Moose project, they already can. And if a user wants to extend it with Moose, they already can. I would also be finding another way to compare arrays besides an Moosified Array::Compare.
Download (untitled) / with headers
text/plain 534b
I'm not going to roll back the Moose changes. I've read carefully everything that has been written on this subject over the last couple of weeks and I'm happy that I'm doing the right thing by switching the module to using Moose. If you want (currently) non-Moose alternatives then please look at List::Compare or Data::Compare. Alternatively, the source code for Array::Compare is available on github[1] - feel free to fork an older version and release a ::Tiny version. Cheers, Dave... [1] http://github.com/davorg/array-compare
Download (untitled) / with headers
text/plain 809b
On 2009-09-08 10:17:13, DAVECROSS wrote: Show quoted text
> I'm not going to roll back the Moose changes. I've read carefully > everything that has been written on this subject over the last couple of > weeks and I'm happy that I'm doing the right thing by switching the > module to using Moose. > > If you want (currently) non-Moose alternatives then please look at > List::Compare or Data::Compare. Alternatively, the source code for > Array::Compare is available on github[1] - feel free to fork an older > version and release a ::Tiny version. > > Cheers, > > Dave... > > [1] http://github.com/davorg/array-compare
I see that List::Compare finally switched to Moo. So actually this request was accepted, and maybe should be turned into a "resolved" instead of "rejected"? Or is Moo also regarded as too heavy-weight?


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.