Closed
Bug 560284
Opened 16 years ago
Closed 16 years ago
*All* errors that checksetup.pl throws should be colorized
Categories
(Bugzilla :: Installation & Upgrading, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.0
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file)
|
8.05 KB,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
Right now there are a lot of errors, like the foreign key errors, that can too-easily be missed by users who are upgrading or installing their Bugzilla. They should be in red, just like the requirement errors.
|
Assignee | |
Comment 1•16 years ago
|
||
Okay, I accomplished this by creating a __DIE__ handler that colorizes its messages if we're not in an eval, and installing it during init_console, and then changing all errors to use "die" instead of print/exit.
|
|
Assignee | |
Updated•16 years ago
|
Flags: approval+
|
|
Assignee | |
Comment 2•16 years ago
|
||
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Constants.pm
modified Bugzilla/DB.pm
modified Bugzilla/DB/Mysql.pm
modified Bugzilla/Install/DB.pm
modified Bugzilla/Install/Requirements.pm
modified Bugzilla/Install/Util.pm
Committed revision 7131.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•