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)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: jussi, Assigned: gerv)

References

Details

Attachments

(1 file)

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:
Attached patch fix the #! lineSplinter Review
Attachment #126767 - Flags: review?(gerv)
We should also enhance the test suite to catch this (and make it flame [error]).
Status: UNCONFIRMED → NEW
Ever confirmed: true
goodperl sounds like a good place for it.
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+
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?
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 2.18
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
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.
Blocks: 211188
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

Creator:
Created:
Updated:
Size: