288539 - minimum DBD::mysql version requirement may need to be increased beyond 2.1028

Status: RESOLVED FIXED

(Bugzilla :: Administration, task)

Description

[Albert Ting]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5) Gecko/20041223 Firefox/1.0
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5) Gecko/20041223 Firefox/1.0

I was not able to get checksetup.pl to run cleanly with the latest bugzilla tip.
 The script now uses the new DB::bz_server_version function.

Was able to solve this problem by upgrading my DBD::mysql version from 2.1028 to
the latest (2.9004)

Reproducible: Always

Steps to Reproduce:

Comment 1

[Max Kanat-Alexander]

No, only to 2.9002.

Comment 2

[Albert Ting]

Attachment: patch

Ok, 2.9002 it is.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Can bz_server_version be hacked to not require the newer version of DBD::mysql?

Comment 4

[Max Kanat-Alexander]

*** Bug 289107 has been marked as a duplicate of this bug. ***

Comment 5

[:glob ✱]

Comment on attachment 179302 [details] [diff] [review]
patch

2.9002 doesn't work on windows, throws:

Renaming indexes...
Undefined subroutine &DBD::mysql::db::SQL_INTEGER called at
d:/Perl/site/lib/DBD/mysql.pm line 362

2.9003 works.

Comment 6

[Dave Miller [:justdave]]

Anyone care to answer Myk's question in comment 3?  Current patch has a denied
review anyway, removing approval request.

Comment 7

[Albert Ting]

(In reply to comment #3)
> Can bz_server_version be hacked to not require the newer version of DBD::mysql?

It could be hacked, but I don't think it would be database independent. 
Granted, I don't know where we're at with respect to Postgres support.

Comment 8

[Max Kanat-Alexander]

We require 2.9003 for many other reasons than bz_server_version. For example,
the new Schema code needs column_info to work properly, and so forth.

2.9003 is *years* old at this point. It's not a big problem, I think.

Comment 9

[Albert Ting]

Attachment: patch v2

Comment 10

[Max Kanat-Alexander]

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.396; previous revision: 1.395
done

Comment 11

[Max Kanat-Alexander]

Added to the release notes in bug 286274.