Closed Bug 143251 Opened 23 years ago Closed 23 years ago

RFE: checksetup.pl should report module version in error message

Categories

(Bugzilla :: Administration, task, P3)

2.15
Other
Linux

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: dkl, Assigned: dkl)

Details

Attachments

(1 file)

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.
Comment on attachment 82922 [details] [diff] [review] Patch to display module version when required module not found (v1) 2xr= justdave
Attachment #82922 - Flags: review+
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: justdave → dkl
Keywords: patch
Target Milestone: --- → Bugzilla 2.16
Wow that was quick. I will get it checked in shortly.
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: 23 years ago
Priority: -- → P3
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: