Closed
Bug 74067
Opened 23 years ago
Closed 23 years ago
run-mozilla.sh -d is broken
Categories
(SeaMonkey :: Build Config, enhancement, P5)
Tracking
(Not tracked)
VERIFIED
FIXED
Future
People
(Reporter: db, Assigned: cls)
Details
The script run-mozilla.sh in the nightly build is broken. I was going to start
it in the debugger bug wrote -d instead of -g. And then the script got into a loop.
> run-mozilla.sh
shift: shift count must be <= $#
shift: shift count must be <= $#
shift: shift count must be <= $#
...
...
Comment 2•23 years ago
|
||
what nightly is this?
Any linux version i've tried. For example the latest nightly build 2001033121
Status: UNCONFIRMED → RESOLVED
Closed: 23 years ago
Resolution: --- → INVALID
-d expects an argument. Hence the shift. Not sure why this is considered a bug.
Well, if you don't give the argument it loops forever. It's marked as a minor bug just because it's not really important and it do work if you give the argument.
Updated•23 years ago
|
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Comment 8•23 years ago
|
||
reopening, setting to future. this is a valid bug, just not important. A simple fix would be to check $# before doing the shift a la: if [$# -ge 2]; then shift 2 else echo "-d requires an argument" exit 1 fi or something like that.
Updated•23 years ago
|
Severity: minor → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P5
Target Milestone: --- → Future
Comment 9•23 years ago
|
||
Is it really minor to have a bunch of these zombie processes running on a machine. I seem to collect them somehow, and can only kill them with -9. Or is that a different issue?
Comment 10•23 years ago
|
||
@@ -290,7 +290,12 @@ ;; -d | --debugger) moz_debugger=$2; - shift 2 + if [ "${moz_debugger}" != "" ]; then + shift 2 + else + echo "-d requires an argument" + exit 1 + fi ;; *) break;
Comment 11•23 years ago
|
||
Patch checked in.
Status: NEW → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Comment 12•23 years ago
|
||
Verified working as intended on Solaris 8 Sparc latest cvs pull as of 11/15/01
Comment 13•23 years ago
|
||
Verified working as intended on Solaris 8 Sparc latest cvs pull as of 11/15/01
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•