Closed Bug 237508 Opened 21 years ago Closed 21 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: 21 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: