Move perl-version-checking into a module

RESOLVED WONTFIX

Status

()

Bugzilla
Bugzilla-General
--
enhancement
RESOLVED WONTFIX
13 years ago
13 years ago

People

(Reporter: Max Kanat-Alexander, Assigned: Max Kanat-Alexander)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

13 years ago
This is a tiny, tiny little part of bug 281494, just to get the initial
structure established.

Basically, this creates the Requirements module, and the Requirements module
requires the correct version of perl. It will also eventually contain a LOT of
functions that are now in checksetup's BEGIN block.
(Assignee)

Updated

13 years ago
Summary: Move perl-version-checking into a subroutine in a module → Move perl-version-checking into a module
(Assignee)

Comment 1

13 years ago
Created attachment 173732 [details] [diff] [review]
Create Bugzilla::Requirements

OK, here it is! :-) As soon as we get this in, I can do more work on checksetup
in this direction.

This is just the *absolute* most tiny change that I can make, just to put this
structure in place.
Attachment #173732 - Flags: review?
(Assignee)

Comment 2

13 years ago
Created attachment 173733 [details] [diff] [review]
Version 2 (use -p1)

Glob pointed out that we really need to do any version checking before we ever
"use" anything -- what if people are using perl 5.0?
Attachment #173732 - Attachment is obsolete: true
Attachment #173733 - Flags: review?
(Assignee)

Updated

13 years ago
Attachment #173732 - Flags: review?
(Assignee)

Comment 3

13 years ago
Actually, this is really too small of a change, and I don't think it's worth it.
I'll start the creation of Requirements.pm from some other code change. But I'll
use some of the stuff that I wrote here as a base.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → WONTFIX

Comment 4

13 years ago
Comment on attachment 173733 [details] [diff] [review]
Version 2 (use -p1)

As you marked this bug as wontfix, I guess I can remove your request for
review. :)
Attachment #173733 - Flags: review?
You need to log in before you can comment on or make changes to this bug.