Closed
Bug 143251
Opened 22 years ago
Closed 22 years ago
RFE: checksetup.pl should report module version in error message
Categories
(Bugzilla :: Administration, task, P3)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: dkl, Assigned: dkl)
Details
Attachments
(1 file)
2.58 KB,
patch
|
justdave
:
review+
justdave
:
review+
|
Details | Diff | Splinter Review |
When checksetup.pl reports that a required module is missing from the system, it should preferrably also report the minimum required version of that module. For example, I have Template-Toolkit installed on my system yet when I ran checksetup.pl it merely said that I need to install the Template-Toolkit which in my mind I already have. Except mine was 2.06 and it was looking for 2.07. Currently it only maintains a array called @missing which contains module names of missing modules. We would need to change this to also maintain a list of versions also mapped to the proper module. So we could change @missing to %missing with the module name as the key and the version as the value. I will make a patch and attach to this report.
Assignee | ||
Comment 1•22 years ago
|
||
Comment 2•22 years ago
|
||
Comment on attachment 82922 [details] [diff] [review] Patch to display module version when required module not found (v1) 2xr= justdave
Attachment #82922 -
Flags: review+
Comment 3•22 years ago
|
||
This is a good thing to have for 2.16 especially in light of the TT2 version. Let's get it checked in (we're just in time for RC, which is happening later tonight). We already branched, check it in both on the trunk and on BUGZILLA-2_16-BRANCH please.
Assignee | ||
Comment 4•22 years ago
|
||
Wow that was quick. I will get it checked in shortly.
Assignee | ||
Comment 5•22 years ago
|
||
BUGZILLA-2_16-BRANCH Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.149.2.1; previous revision: 1.149 done HEAD Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.150; previous revision: 1.149 done RESOLVED/FIXED
Status: NEW → RESOLVED
Closed: 22 years ago
Priority: -- → P3
Resolution: --- → FIXED
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•