Bug #11584 for CPANPLUS: Prereq versions in FAIL reports give incorrect version number

Sun Feb 20 11:09:37 2005 Guest - Ticket created

Subject:

Prereq versions in FAIL reports give incorrect version number

For example, see http://www.nntp.perl.org/group/perl.cpan.testers/185517 Path::Class 0.08 was installed, but it gave version 0. There appears to be nothing unusual in Path::Class, except that the $VERSION assignment lacks a "my" or "our". See http://search.cpan.org/src/KWILLIAMS/Path-Class-0.08/lib/Path/Class.pm However, VERSION was not exported, so it might be a problem with Path::Class, and not with CPANPLUS.

Wed Feb 23 07:50:53 2005 kane [...] cpan.org - Correspondence added

[guest - Sun Feb 20 11:09:37 2005]: Show quoted text

> For example, see > http://www.nntp.perl.org/group/perl.cpan.testers/185517 > > Path::Class 0.08 was installed, but it gave version 0. > > There appears to be nothing unusual in Path::Class, except that the > $VERSION assignment lacks a "my" or "our". See > http://search.cpan.org/src/KWILLIAMS/Path-Class-0.08/lib/Path/Class.pm > > However, VERSION was not exported, so it might be a problem with > Path::Class, and not with CPANPLUS.

Hmm, looking at the module, the version should be parsed just fine. What does your output look like if you do this? CPAN Terminal> l Path::Class Details for 'Path::Class' Author Ken Williams (ken@mathforum.org) Description None given Development Stage Unknown Interface Style Unknown Language Used Unknown Package Path-Class-0.08.tar.gz Public License Unknown Support Level Unknown Version Installed 0.08 Version on CPAN 0.08 Contains: Path::Class Path::Class::Dir Path::Class::Entity Path::Class::File As you can see, for me it parses correctly (and the same code is used to prepare the test report). So i can't quite reproduce this problem i'm afraid.

Wed Feb 23 07:50:54 2005 kane [...] cpan.org - Status changed from 'new' to 'stalled'

Fri Jun 03 08:26:49 2005 kane [...] cpan.org - Correspondence added

Sorry, this reply slipped through the cracks of my mailbox... [RRWO - Wed Feb 23 10:48:07 2005]: Show quoted text

> I get the following output. I'm not sure why you're having that > problem.

Ehm, i'm not having a problem -- thought you reported a bug here... However, your output looks identical to mine. Show quoted text

>

> > Details for 'Path::Class' > > Author Ken Williams (ken@mathforum.org) > > Description None given > > Development Stage Unknown > > Interface Style Unknown > > Language Used Unknown > > Package Path-Class-0.08.tar.gz > > Public License Unknown > > Support Level Unknown > > Version Installed 0.08 > > Version on CPAN 0.08 > > Contains: Path::Class > > Path::Class::Dir > > Path::Class::Entity > > Path::Class::File

> > Anyhow, shouldn't it get the version from $module::VERSION, and not > parsing a file?

You can only do that by running the code, which is in most cases when we need the version not possibly, and in almost every case not desirable. Show quoted text

> > As you can see, for me it parses correctly (and the same code is

> used to prepare the test

> > report). So i can't quite reproduce this problem i'm afraid.

So this part is still valid -- i can't reproduce, and your test case doesn't show a reproducable error either. Can you provide an example where the error persists, otherwise i'll mark this ticket as dead.

Fri Jun 03 08:26:50 2005 The RT System itself - Status changed from 'stalled' to 'open'

Fri Jun 24 03:58:40 2005 kane [...] cpan.org - Status changed from 'open' to 'resolved'