718786 - JavaScript shell exits with a zero status when program is terminated

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jim Blandy :jimb]

If the JavaScript shell runs a file (via -f) whose code terminates (that is, execution returns an error, but no exception is set), the shell exits with a zero status. It should only exit with a zero status when all files specified by -f and scripts given using -e complete successfully.

For example:

$ cat foo.js
terminate();
$ cat bar.js
print("Hi, from bar!");
$ js -f foo.js; echo $?
0
$ js -f bar.js; echo $?
Hi, from bar!
0
$ js -f foo.js -f bar.js; echo $?
Hi, from bar!
0
$ 

(Here, 'terminate' is the function defined by the patch in bug 717417.)

Comment 1

[Jim Blandy :jimb]

Attachment: Make the shell exit with EXITCODE_RUNTIME_ERROR when code is terminated.

In js/src/jit-test/jit_test.py, add an 'exitstatus' jit-test cookie
attribute (and rename test.error to test.expect_error). Mark tests that
terminate with their expected termination code.

Clean up TMFLAGS comments.

Comment 2

[Jason Orendorff [:jorendorff]]

Comment on attachment 591676 [details] [diff] [review]
Make the shell exit with EXITCODE_RUNTIME_ERROR when code is terminated.

Thanks!

I'd even be OK with a bigger change in behavior here, where a single failing -e or -f causes Process to return false, and its caller checks the return value and bails out. Separate bug if so.

Comment 3

[Jim Blandy :jimb]

(In reply to Jason Orendorff [:jorendorff] from comment #2)
> I'd even be OK with a bigger change in behavior here, where a single failing
> -e or -f causes Process to return false, and its caller checks the return
> value and bails out. Separate bug if so.

Actually, every caller of Process already checks gExitCode, so that is the behavior this patch brings about.

Comment 4

[Jim Blandy :jimb]

try: https://tbpl.mozilla.org/?tree=Try&rev=6914ca5efbd0

Comment 5

[Jim Blandy :jimb]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ab3edf81a46f

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/ab3edf81a46f