Okay, this is the big one. The whole --TABLE-- section of schema updates is going to be moved into Bugzilla::Install::DB. I'm going to create one function: update_table_definitions. This is going to call a bunch of other private functions, so that it will be easy to read. I may do this in pieces, we'll see.

Okay, it turns out that this patch depends on bug 289357, which requires review. So, in order to avoid some terrible bitrot, I'm only going to move *most* changes into the module, in this patch. I'll leave a lot of the most recent changes in checksetup for now, and that way if there are any checkins before I check in this patch, it won't bitrot horribly.

Okay, here's the patch. Here's what I've done: 1) Created Bugzilla::Install::DB. 2) Created a useful indicate_progress subroutine in there that people should use all the time if they're going to do something that takes a long time. 3) Split up all of the stuff that checksetup does with the tables into individual subroutines that are all called by one main subroutine. 4) Fixed up some of the code that migrates groups from 2.16 so that it inserts users into the admin group directly, instead of waiting for checksetup to insert them into the admin group later.

Requesting approval directly, as module owner, even though the patch isn't ready to check in yet (because it depends on bug 289357 being reviewed and checked in first).

Because of a checkin fix on bug 289357, I had to manually modify the patch before checking it in. I'll attach the fixed patch. Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.533; previous revision: 1.532 done RCS file: /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm,v done Checking in Bugzilla/Install/DB.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm,v <-- DB.pm initial revision: 1.1 done Checking in Bugzilla/Install/Filesystem.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Filesystem.pm,v <-- Filesystem.pm new revision: 1.5; previous revision: 1.4 done