Closed
Bug 317899
Opened 20 years ago
Closed 20 years ago
checksetup needlessly requires a version of MIME::Base64 newer than perl 5.8.5 (RHEL4)
Categories
(Bugzilla :: Installation & Upgrading, defect, P1)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.22
People
(Reporter: mkanat, Assigned: mkanat)
Details
Attachments
(1 file)
|
501 bytes,
patch
|
glob
:
review+
|
Details | Diff | Splinter Review |
checksetup requires MIME::Base64 3.05. I asked glob, and he said that there's no particular reason for the requirement.
RHEL4's perl 5.8.5 only comes with MIME::Base64 3.03. It's a big hassle to upgrade it every time perl gets upgraded. (Since MIME::Base64 is part of the "perl" package in RHEL, upgrades overwrite the MIME::Base64 that you've installed yourself.)
| Assignee | ||
Updated•20 years ago
|
Priority: -- → P1
Version: 2.20 → 2.21
| Assignee | ||
Comment 1•20 years ago
|
||
OK, actually, we were requiring version 3.03, and perl 5.8.5 has version 3.01. Now, there was some funny logic in here that I'm checking, so I just wanted to run this (one-line) patch by you glob, to make sure that I'm not breaking anything...
Attachment #204260 -
Flags: review?(bugzilla)
| Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Comment 2•20 years ago
|
||
| Assignee | ||
Comment 3•20 years ago
|
||
Interesting. I think that since MIME::Base64 actually comes with perl, at least on Red Hat, that particular comment isn't an issue for our requirement (the fact that it fails "make test"). And anybody else who would be installing it currently would be installing the latest version, so that also wouldn't be an issue.
Comment on attachment 204260 [details] [diff] [review]
v1
r=glob
i agree with max, while 3.01 has problems with its test suite, we'll probably get more issues from people upgrading perl's rpm (which downgrades mime::base64).
Attachment #204260 -
Flags: review?(bugzilla) → review+
| Assignee | ||
Updated•20 years ago
|
Flags: approval?
Flags: approval2.20?
Updated•20 years ago
|
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
| Assignee | ||
Comment 5•20 years ago
|
||
Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl
new revision: 1.454; previous revision: 1.453
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•