Closed
Bug 211127
Opened 21 years ago
Closed 21 years ago
chart.cgi uses /usr/bonsaitools/bin/perl instead of /usr/bin/perl
Categories
(Bugzilla :: Reporting/Charting, defect)
Bugzilla
Reporting/Charting
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: jussi, Assigned: gerv)
References
Details
Attachments
(1 file)
457 bytes,
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5a) Gecko/20030630 Build Identifier: chart.cgi uses /usr/bonsaitools/bin/perl instead of /usr/bin/perl in the #! line. Reproducible: Always Steps to Reproduce:
Reporter | ||
Comment 1•21 years ago
|
||
Reporter | ||
Updated•21 years ago
|
Attachment #126767 -
Flags: review?(gerv)
Comment 2•21 years ago
|
||
We should also enhance the test suite to catch this (and make it flame [error]).
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 3•21 years ago
|
||
goodperl sounds like a good place for it.
Assignee | ||
Comment 4•21 years ago
|
||
Comment on attachment 126767 [details] [diff] [review] fix the #! line Oops. r=gerv. If you need this checked in (i.e. you don't have checkin privs), shout. Gerv
Attachment #126767 -
Flags: review?(gerv) → review+
Reporter | ||
Comment 5•21 years ago
|
||
Yes, I do not have cvs write access. So go ahead and checkin. Maybe that test suite thing should be a bug of it's own?
Flags: approval?
Updated•21 years ago
|
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 2.18
Assignee | ||
Comment 6•21 years ago
|
||
Fixed. Thank you :-) Checking in chart.cgi; /cvsroot/mozilla/webtools/bugzilla/chart.cgi,v <-- chart.cgi new revision: 1.2; previous revision: 1.1 done Gerv
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 7•21 years ago
|
||
We should test for this, yea. I'll file a new bug for it when I get around to it. While I'm inclined to say there isn't much point in bothering due to the fact that newinstall will make this all different anyway, the impending doom of newinstall means that I wouldn't hold your breath on it. In other words, I'll file a new bug for testing for /usr/bin/perl.
Updated•11 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•