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

RESOLVED FIXED in Bugzilla 2.18

Status

()

defect
RESOLVED FIXED
16 years ago
7 years ago

People

(Reporter: gerv, Assigned: justdave)

Tracking

2.17.5
Bugzilla 2.18
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(1 attachment)

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
Posted 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 3

15 years ago
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?
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: 15 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.