678772 - version.pm 0.92 and newer forbids negative values, making checksetup.pl fail with the error "Invalid version format"

Status: RESOLVED FIXED

(Bugzilla :: Installation & Upgrading, defect)

Description

[Frédéric Buclin]

Perl 5.15.1 has version.pm 0.93, but ActiveState already offers it as an update via the Perl Package Manager (ppm) for at least Perl 5.14.1 (I didn't check for 5.12.4). Also, Linux distros could do the same if they want to (and some probably do). Since version.pm 0.92, negative values for $VERSION are no longer allowed, and an error is thrown if either $VERSION is negative or you try to compare $VERSION to a negative value. In this last case, the reason is that Perl automagically converts the negative value into a version object, but as version.pm now forbids them, an error is thrown. This makes checksetup.pl to die due to line 712 in Bugzilla::Install::Requirements::_checking_for():

  if ($found and $found eq "-1")

$found = $module->VERSION || -1; and so is a version object when the module being called returns a version object (most Perl modules still return a string, but some others return a version object, such as DBD::Pg). In that case, "-1" is converted into an illegal (negative) version object and checksetup.pl dies:

"Invalid version format (non-numeric data) at Bugzilla/Install/Requirements.pm line 712."

IMO, we shouldn't try to set $found to -1 when the module cannot be found. $found should simply be undefined. I know _checking_for() currently uses undefined for "module_unknown_version", but I doubt such modules exist (probably CPAN has a check to reject the upload of such modules).

Comment 1

[Frédéric Buclin]

If you want to reproduce the error with version.pm 0.92 or newer, simply run this testcase from the shell:

perl -wE 'package Foo; use version; our $VERSION = qv("0.89.2"); my $ver = Foo->VERSION; say $ver unless $ver eq "-1"'

Comment 2

[Frédéric Buclin]

Attachment: patch, v1

If it's a reference, then it's a version object. In that case, it cannot be -1.

Comment 3

[Max Kanat-Alexander]

Comment on attachment 552915 [details] [diff] [review]
patch, v1

Review of attachment 552915 [details] [diff] [review]:
-----------------------------------------------------------------

Looks great.

Comment 4

[Frédéric Buclin]

Committing to: bzr+ssh://[e-mail]/bugzilla/trunk/
modified Bugzilla/Install/Requirements.pm
Committed revision 7923.

Committing to: bzr+ssh://[e-mail]/bugzilla/4.2/
modified Bugzilla/Install/Requirements.pm
Committed revision 7903.

Committing to: bzr+ssh://[e-mail]/bugzilla/4.0/
modified Bugzilla/Install/Requirements.pm
Committed revision 7644.