65598 - checksetup.pl should check minimum versions on prerequisites

Status: VERIFIED FIXED

(Bugzilla :: Bugzilla-General, enhancement)

Description

[Dave Miller [:justdave]]

As was discovered in bug 52921, checksetup.pl is not checking for the minimum 
required versions of the various prerequisites.  It merely checks to see if you 
have that module/program, it doesn't make sure it's a recent enough version.

The module checking section in checksetup.pl should include checks for the 
version numbers for modules that matter.  It wouldn't hurt if it would check the 
version of mySQL and Sendmail as well, instead of printing the reminder notice 
all the time at the end.

The majority of the Perl modules' versions can be checked by looking at the 
$ModuleName::VERSION variable after the module has been loaded.

mySQL version can be checked by logging in and doing SELECT VERSION()

Sendmail version can be checked by doing "/usr/sbin/sendmail -bs" and parsing the 
version number out of the 220 reponse line, then closing the pipe.

As discovered in bug 52921, minimum version for DBI is 1.13 to use Bugzilla's 
current features.

Comment 1

[Dave Miller [:justdave]]

Attachment: Check version of DBI

Comment 2

[Dave Miller [:justdave]]

The version comparison used in the above patch will pass version 1.5 (which it 
shouldn't, because 1.13 is newer).  So it'll need to have the logic redone a 
little.

Comment 3

[Dave Miller [:justdave]]

Yeah, yeah, I know we're trying to eliminate the 2.12 stuff, but this really 
needs to be in there. :)  Especially since we have stuff in the code that 
requires a newer version of DBI than what b.m.o. is running :-)

I'm taking ownership on this one, will attempt to have a more comprehensive patch 
up in the next day or two.

Comment 4

[Dave Miller [:justdave]]

OK, just need to know what version numbers to use for all the Perl modules...

The README doesn't mention most of them.  I have a system I installed Bugzilla on 
about a year ago that I know most of the Perl stuff hasn't been updated since 
then...  should I just take whatever the version numbers are on there and assume 
that's what we need?

Terry, you've been around Bugzilla the longest, you happen to remember what it 
requires, or do we just know we need that module?

Comment 5

[Dave Miller [:justdave]]

Attachment: Patch to check versions

Comment 6

[Dave Miller [:justdave]]

OK, here's the patch.  It does everything but Sendmail, and I'm out of time to 
add that part before making a trip this afternoon.  If I get around to it, maybe 
I'll add that part later tonight.  It checks versions the correct way (3.22.32 is 
greater than 3.22.5) and also checks the version of the MySQL server.

For the Perl modules that I couldn't find minimum versions listed for anywhere, 
the check still just checks to make sure you have it, and doesn't care what 
version.

If I don't get the sendmail part in, it can still get into 2.12 as is.

Comment 7

[Dave Miller [:justdave]]

per irc, r=tara



checked in.



We can file a new bug to check for Sendmail if we really want it.

Comment 8

[Tara Hernandez]

Sorry for the spam, but I needed to be able to query for all of these correctly.

Comment 9

[Jacob Steenhagen]

V.  This is much nicer then the simple "Reminder:" lines at the end.  :)

Comment 10

[Dave Miller [:justdave]]

Moving closed bugs to Bugzilla product