Bug #56740 for Module-CoreList: Format Perl versions >= 5.6.0 as X.Y.Z

Tue Apr 20 04:06:42 2010 mschwern [...] cpan.org - Ticket created

Subject:

Format Perl versions >= 5.6.0 as X.Y.Z

Decimal style Perl version numbers are increasingly awkward. 5.01 or 5.010000 is hard to equate with 5.10.0. It would be easier to read if Module::CoreList formatted version numbers from 5.6 onward as dotted versions to match how they are presented elsewhere.

Tue Apr 20 07:16:01 2010 mschwern [...] cpan.org - Correspondence added

I've fixed this, along with normalizing the display of versions in corelist. You can see the changes and patches here: http://github.com/schwern/Module-Corelist/commits/master

Tue Apr 20 07:16:19 2010 mschwern [...] cpan.org - Status changed from 'new' to 'open'

Tue Apr 20 07:25:26 2010 mschwern [...] cpan.org - Correspondence added

PS The test t/corelist-format.t is sensitive to the data in Module::CoreList. So it is useful, but will have to be changed when the data about Perl versions changes. That's probably simpler than trying to come up with something generic or generated from the data. Anyhow, I needed something to prove the code correct and provide a regression. Your choice if you want to keep it.

Tue Apr 20 09:40:40 2010 RGARCIA [...] cpan.org - Taken

Tue Apr 20 09:42:32 2010 RGARCIA [...] cpan.org - Correspondence added

Thanks, I've merged your changes in bleadperl, so they'll be in the version of Module::Corelist released along 5.13.0. I haven't kept the test file, because it doesn't run in the core as-is (Test::Differences is not a core module), although it's a good idea and I might adapt it later.

Tue Apr 20 09:42:33 2010 RGARCIA [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Apr 20 10:21:08 2010 schwern [...] pobox.com - Correspondence added

Subject:	Re: [rt.cpan.org #56740] Format Perl versions >= 5.6.0 as X.Y.Z
Date:	Tue, 20 Apr 2010 17:20:49 +0300
To:	bug-Module-CoreList [...] rt.cpan.org
From:	Michael G Schwern <schwern [...] pobox.com>

Rafaël Garcia-Suarez via RT wrote: Show quoted text

> Thanks, I've merged your changes in bleadperl, so they'll be in the version of Module::Corelist > released along 5.13.0.

Cool. Show quoted text

> I haven't kept the test file, because it doesn't run in the core as-is (Test::Differences is not a > core module), although it's a good idea and I might adapt it later.

You can just switch that from eq_or_diff() to is(). TD just offers better diagnostics. -- On error resume stupid

Tue Apr 20 10:21:10 2010 The RT System itself - Status changed from 'resolved' to 'open'

Wed Feb 09 10:17:01 2011 BINGOS [...] cpan.org - Correspondence added 1 min

Closing ticket.

Wed Feb 09 10:17:02 2011 BINGOS [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #56740 for Module-CoreList: Format Perl versions >= 5.6.0 as X.Y.Z

Preferred bug tracker