Closed Bug 344055 Opened 18 years ago Closed 18 years ago

Bugzilla now requires Filter::Util::Call

Categories

(Bugzilla :: Installation & Upgrading, defect)

2.23
defect
Not set
major

Tracking

()

RESOLVED WONTFIX

People

(Reporter: mkanat, Assigned: mkanat)

Details

Attachments

(1 file)

The way that I use Switch in Bugzilla::Auth requires that people have the Filter::Util::Call module installed. For some reason this doesn't seem to ship in the "perl" package, at least in Red Hat, even though Switch requires it, and Switch ships with perl.
Attached patch v1Splinter Review
Attachment #228628 - Flags: review?(wicked+bz)
Status: NEW → ASSIGNED
Comment on attachment 228628 [details] [diff] [review]
v1

Ok, seems to work on Linux even though name doesn't exactly fit in the column. :)
Attachment #228628 - Flags: review?(wicked+bz) → review+
Flags: approval?
Flags: approval? → approval+
It's confusing to check for something Bugzilla doesn't use directly. I would say it's a Perl problem if Filter::Util::Call is missing while Switch.pm is available.
Okay. Yeah, actually, I agree with LpSolit. You can see that Filter::Util::Call is perldoc'ed as part of base perl:

http://perldoc.perl.org/Filter/Util/Call.html

(CPAN-only modules don't show up on that site.)

So that means that somebody's been messing with their installation if they don't have Filter::Util::Call, and thus we don't need to add this check.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Flags: approval+
Resolution: --- → WONTFIX
Target Milestone: Bugzilla 2.24 → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: