From: David Golden [mailto:[EMAIL PROTECTED] > > [Resent from my subscribed address:] > > David Cantrell raises this edge case in CPAN::Reporter: > > > The distribution name 'Sendmail_M4.0.27.tar.gz' does not appear to > > be > > formatted according to CPAN tester guidelines. Perhaps it is not a > > normal CPAN distribution. > > How do you think we should handle this case? Right now CPAN::Reporter > doesn't send reports unless the distribution tarball matches: > > qr/(.+)([\-\_])(v?\d.*)(\.(?:tar\.(?:gz|bz2)|tgz|zip))$/; > > This was added at Barbie's suggested based on the use of this regex in > YACSmoke.
This was the regex that was previously used by the cpansmoke script that came with early versions (pre 0.50) of CPANPLUS. > We could make the separator optional: > > qr/(.+)([\-\_]?)(v?\d.*)(\.(?:tar\.(?:gz|bz2)|tgz|zip))$/; That would break every distribution that ends with a number, e.g. Win32-0.31.tar.gz (taunts not withstanding ;)). > Alternatively, we could just confirm that the distribution appears to > be a tarball and hope that's a valid distribution and that the CPAN > Testers architecture can parse out a dist-name and version. My take is that the distribution is badly named, and I usually email the author to advise of them of that, explaining that although they are free to upload with their own naming conventions, the CPAN Testers mechanism will not be guaranteed to test their distribution if they use a format that does match the convention. We should stick to a recommended convention, and edge cases accept that they are not going to get tested. Cheers, Barbie. -- Birmingham Perl Mongers - http://birmingham.pm.org Miss Barbell Productions - http://www.missbarbell.co.uk
