Bug #97628 for Module-CPANTS-Analyse: Proposed new metric: distname in release filename doesn't match dist name in metadata

Wed Jul 30 14:30:41 2014 NEILB [...] cpan.org - Ticket created

Subject:

Proposed new metric: distname in release filename doesn't match dist name in metadata

Hi Ishigaki-san, What do you think about a new metric which checks whether the distname in the file name matches the distname given in the dist's metadata? Ie if the distname returned by CPAN::DistnameInfo($release_path)->dist ne metadata->{name} Lots of tools use CPAN::DistnameInfo to get the dist name for a release, but in some situations the path might not be easily available. In some cases I've used the MetaCPAN API to get the dist's dist name, but in some cases they don't match. I'm slowly working through those, raising bugs, but it would be good to have a check for that! Cheers, Neil

Wed Jul 30 18:45:19 2014 ether [...] cpan.org - Correspondence added

On 2014-07-30 11:30:41, NEILB wrote: Show quoted text

> Hi Ishigaki-san, > > What do you think about a new metric which checks whether the distname > in the file name matches the distname given in the dist's metadata?

+1

Wed Jul 30 18:45:19 2014 The RT System itself - Status changed from 'new' to 'open'

Wed Jul 30 23:36:26 2014 ishigaki [...] cpan.org - Correspondence added

Implemented in the master, and started regenerating CPANTS databases. Confirmed ~1% of minicpan failed. Let's see what happens for the whole CPAN/BackPAN. Thanks! https://github.com/cpants/Module-CPANTS-SiteKwalitee/commit/7f9102563611df98240d1ae366930cdc8723664e On Thu Jul 31 03:30:41 2014, NEILB wrote: Show quoted text

> Hi Ishigaki-san, > > What do you think about a new metric which checks whether the distname > in the file name matches the distname given in the dist's metadata? > > Ie if the distname returned by CPAN::DistnameInfo($release_path)->dist > ne metadata->{name} > > Lots of tools use CPAN::DistnameInfo to get the dist name for a > release, but in some situations the path might not be easily > available. In some cases I've used the MetaCPAN API to get the dist's > dist name, but in some cases they don't match. I'm slowly working > through those, raising bugs, but it would be good to have a check for > that! > > Cheers, > Neil

Wed Jul 30 23:36:27 2014 ishigaki [...] cpan.org - Status changed from 'open' to 'patched'

Thu Jul 31 03:09:31 2014 NEILB [...] cpan.org - Correspondence added

素晴らしい！

Thu Jul 31 22:40:03 2014 ishigaki [...] cpan.org - Correspondence added

Done and ready. Thanks. http://cpants.cpanauthors.org/kwalitee/distname_matches_name_in_meta On Thu Jul 31 12:36:26 2014, ISHIGAKI wrote: Show quoted text

> Implemented in the master, and started regenerating CPANTS databases. > Confirmed ~1% of minicpan failed. Let's see what happens for the whole > CPAN/BackPAN. Thanks! > > https://github.com/cpants/Module-CPANTS- > SiteKwalitee/commit/7f9102563611df98240d1ae366930cdc8723664e > > On Thu Jul 31 03:30:41 2014, NEILB wrote:

> > Hi Ishigaki-san, > > > > What do you think about a new metric which checks whether the > > distname > > in the file name matches the distname given in the dist's metadata? > > > > Ie if the distname returned by CPAN::DistnameInfo($release_path)-

> > >dist

> > ne metadata->{name} > > > > Lots of tools use CPAN::DistnameInfo to get the dist name for a > > release, but in some situations the path might not be easily > > available. In some cases I've used the MetaCPAN API to get the dist's > > dist name, but in some cases they don't match. I'm slowly working > > through those, raising bugs, but it would be good to have a check for > > that! > > > > Cheers, > > Neil

Thu Jul 31 22:40:05 2014 ishigaki [...] cpan.org - Status changed from 'patched' to 'resolved'