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)
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•23 years ago
|
||
Comment 2•23 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•23 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•23 years ago
|
||
Wow that was quick. I will get it checked in shortly.
| Assignee | ||
Comment 5•23 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: 23 years ago
Priority: -- → P3
Resolution: --- → FIXED
Updated•13 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
•