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)

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: 22 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: