Closed Bug 347083 Opened 19 years ago Closed 19 years ago

Move the rest of checksetup's DB-modification code into a module

Categories

(Bugzilla :: Installation & Upgrading, enhancement)

2.23
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

So, I've moved most of the schema-updating code into Bugzilla::Install::DB. Now I just have to remove the rest. This patch will probably bitrot any patch that currently adds or modifies a column, so I'm CC'ing everybody who might be working on an affected patch that I can think of. It should be easy to unbitrot, most things will just get moved right into Bugzilla::Install::DB.
Attached patch v1Splinter Review
Okay, here it is! All the schema-modification code is now in Bugzilla::Install::DB. The file is commented extensively, if you're interested in how best to modify it.
Assignee: installation → mkanat
Status: NEW → ASSIGNED
Requesting approval directly as module owner.
Flags: approval?
Flags: approval? → approval+
Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.534; previous revision: 1.533 done Checking in Bugzilla/Install/DB.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm,v <-- DB.pm new revision: 1.2; previous revision: 1.1 done Checking in Bugzilla/Install/Filesystem.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Filesystem.pm,v <-- Filesystem.pm new revision: 1.6; previous revision: 1.5 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: