Checksetup can't find MIME::Tools with Perl 5.8.0

RESOLVED FIXED in Bugzilla 2.22

Status

()

P1
major
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: bugreport, Assigned: bugreport)

Tracking

2.21
Bugzilla 2.22
Bug Flags:
approval +

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

14 years ago
MIME::Tools cannot be found by the checksetup version checks on installations
with Perl 5.8.0

If the reference to MIME::Tools is changed to MIME::Parser, the exact correct
version number is correctly returned.
maybe we need to additionally check for $ModuleName::version like CPAN does...
(Assignee)

Comment 2

14 years ago
I tried that.  It still doesn't help.
The problem is that $@ gets set if you require MIME::Tools instead of MIME::Parser
(Assignee)

Comment 3

14 years ago
Created attachment 191693 [details] [diff] [review]
Change Tools to Parser
(Assignee)

Updated

14 years ago
Attachment #191693 - Flags: review?
Comment on attachment 191693 [details] [diff] [review]
Change Tools to Parser

This doesn't work for me (ActiveState perl on Windows). I'm told I should run
"ppm install MIME::Parser", which yields "Searching for 'MIME::Parser' returned
no results. Try a broader search first."

I've got MIME-tools 5.411a installed (which works with bug 126266 landed if I
trick Bugzilla into thinking 5.411 is sufficient). This package contains
MIME::Parser 5.406.
Attachment #191693 - Flags: review? → review-
(Assignee)

Comment 5

14 years ago
Does this mean we just need to additionally change the hash from....


    'MIME::Tools'       => 'MIME-Tools',

to

    'MIME::Parser'       => 'MIME-Tools',
(Assignee)

Comment 6

14 years ago
Created attachment 191701 [details] [diff] [review]
Revised to have ppm suggest correct name as well

This should do the trick, but I cannot test it on ASPN.
Attachment #191693 - Attachment is obsolete: true
Attachment #191701 - Flags: review?
(Assignee)

Updated

14 years ago
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.22
Well, that looks better, checksetup.pl wants me to update MIME-tools to 5.417
now. When I |ppm install MIME-Tools|, I get told "Note: Package 'MIME-tools' is
already installed." A newer version of MIME-tools than 5.411a seems not to be
available via ppm at the moment...

I'm not so sure that MIME::Parser is the right source for the version number...
For my MIME-tools version 5.411a, MIME::Parser has version 5.406.
(Assignee)

Comment 8

14 years ago
Glob: Is MIME::Parser 5.406 recent enough???
(In reply to comment #8)
> Glob: Is MIME::Parser 5.406 recent enough???

this patch was developed on MIME::Parser 5.406 (which is in MIME::Tools 5.417
and 5.411).

Comment 10

14 years ago
(In reply to comment #9)
> this patch was developed on MIME::Parser 5.406 (which is in MIME::Tools 5.417
> and 5.411).

Some users on IRC complained they couldn't find MIME::Tools 5.417 on Windows. I
told them to hack checksetup.pl to require MIME::Tools 5.411.

glob, if MIME::Parser 5.406 is in MIME::Tools 5.411, why requiring 5.417?
Couldn't we downgrade the minimum requirement to 5.411? Per Marc's comment 4 and
comment 7, there is no problem doing so.
> Couldn't we downgrade the minimum requirement to 5.411? Per Marc's comment 4 and
> comment 7, there is no problem doing so.

i have no issues with this.
(Assignee)

Comment 12

14 years ago
Comment on attachment 191701 [details] [diff] [review]
Revised to have ppm suggest correct name as well

OK then... we'll check MIME::Parser for minimum version 5.406 and instruct PPM
users to install MIME-Tools while instructing CPAN users to install
MIME::Parser which, according to
http://search.cpan.org/~dskoll/MIME-tools-5.417/lib/MIME/Parser.pm will install
MIME::Tools
Attachment #191701 - Attachment is obsolete: true
Attachment #191701 - Flags: review?
(Assignee)

Comment 13

14 years ago
Created attachment 191886 [details] [diff] [review]
updated version numbers

Also fixed the mapping from MIME::Base64 to suggest the MIME-Base64 name for
the ASPN module.
Attachment #191886 - Flags: review?
(Assignee)

Updated

14 years ago
Attachment #191886 - Flags: review? → review?(bugzilla)
Comment on attachment 191886 [details] [diff] [review]
updated version numbers

r=glob
Attachment #191886 - Flags: review?(bugzilla) → review+
(Assignee)

Updated

14 years ago
Flags: approval?
Flags: approval? → approval+
Comment on attachment 191886 [details] [diff] [review]
updated version numbers

r=wurblzap.
Attachment #191886 - Flags: review+

Updated

14 years ago
Assignee: installation → bugreport
Hardware: PC → All
(Assignee)

Comment 16

14 years ago
Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.424; previous revision: 1.423
done
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
OS: Linux → All
You need to log in before you can comment on or make changes to this bug.