Closed Bug 237508 Opened 20 years ago Closed 20 years ago

checksetup.pl should use $^X when telling admin to install modules

Categories

(Bugzilla :: Installation & Upgrading, defect)

2.17.5
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: gerv, Assigned: justdave)

References

Details

Attachments

(1 file)

In .webtools, Dave said:

> The output from checksetup.pl when it gives instructions should probably
> use $^X so no matter what perl they're using it tells them to use that same
> one to install the modules.

Sounds like a good plan to me (this helps to mitigate the double-Perl-version
problem.)

Gerv
Attached patch Patch v1Splinter Review
rationale for Chart::Base install change: we used to specify 0.99b because 1.01
was broken.  Current version is 2.x, and works fine, and it makes this patch
easier. :)
Assignee: zach → justdave
Status: NEW → ASSIGNED
Comment on attachment 143941 [details] [diff] [review]
Patch v1

need review from a win32 type, since I'm touching the Chart::Base install
instructions on win32
Attachment #143941 - Flags: review?(jouni)
Comment on attachment 143941 [details] [diff] [review]
Patch v1

AFAICT, this is as good as it gets. However, as noted on developers@ earlier, I
don't think there is a Chart::Base PPD available for 5.8. It might be worth
relnoting?
Attachment #143941 - Flags: review?(jouni) → review+
Flags: approval?
Blocks: 239143
Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.271; previous revision: 1.270
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Flags: approval? → approval+
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 2.18
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: