Skip Menu |
 

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

Report information
The Basics
Id: 105697
Status: open
Priority: 0/
Queue: Module-CPANTS-Analyse

People
Owner: Nobody in particular
Requestors: ether [...] cpan.org
Cc:
AdminCc:

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



Subject: new kwalitee metric: distribution version formatting
Download (untitled) / with headers
text/plain 525b
I was recently talking to the macports folks who create distribution port files for perl modules for macports. They say "version formatting is sometimes an issue for us - particularly versions that use alpha characters (prepending a "v" for instance)." We can easily check for this with CPANTS -- so I propose that we add a metric to check that the distribution version (as in the "version" meta field, as well as the tarball filename) not only passes a version::is_lax() check, but also contains no alpha characters at all.
Download (untitled) / with headers
text/plain 971b
On Tue Jul 07 06:47:47 2015, ETHER wrote: Show quoted text
> I was recently talking to the macports folks who create distribution > port files for perl modules for macports. They say "version formatting > is sometimes an issue for us - particularly versions that use alpha > characters (prepending a "v" for instance)." > > We can easily check for this with CPANTS -- so I propose that we add a > metric to check that the distribution version (as in the "version" > meta field, as well as the tarball filename) not only passes a > version::is_lax() check, but also contains no alpha characters at all.
I'm not sure how much this helps other people. If we should always warn an alphabet in a dist version including a prepending "v" (4605 distributions on BackPAN) or even "-TRIAL" (1431), that's fine and we can make has_proper_version in ::SiteKwalitee::DistVersion a bit more strict. If not, and this only helps macports people, I'd rather ask/help them to fix their tools. Thoughts?
Download (untitled) / with headers
text/plain 752b
On 2015-07-06 20:53:29, ISHIGAKI wrote: Show quoted text
> > I'm not sure how much this helps other people. If we should always > warn an alphabet in a dist version including a prepending "v" (4605 > distributions on BackPAN) or even "-TRIAL" (1431), that's fine and we > can make has_proper_version in ::SiteKwalitee::DistVersion a bit more > strict. If not, and this only helps macports people, I'd rather > ask/help them to fix their tools. > > Thoughts?
I think we (Toolchain Gang) should write up a document that defines best practices for versions, which clearly describes the caveats of each type of version format. I've heard that the "v1.2.3" format is more harmful than "1.2.3", but I am not able to quantify the difference. Hopefully someone else can! :)


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.