Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Module-CoreList CPAN distribution.

Report information
The Basics
Id: 56740
Status: resolved
Worked: 1 min
Priority: 0/
Queue: Module-CoreList

People
Owner: RGARCIA [...] cpan.org
Requestors: mschwern [...] cpan.org
Cc:
AdminCc:

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



Subject: Format Perl versions >= 5.6.0 as X.Y.Z
Download (untitled) / with headers
text/plain 262b
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.
Download (untitled) / with headers
text/plain 177b
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
Download (untitled) / with headers
text/plain 383b
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.
Download (untitled) / with headers
text/plain 288b
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.
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>
Download (untitled) / with headers
text/plain 460b
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
Closing ticket.


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.