Paths

Table of Contentst

Differential D11456

Fix Aphlict exit status
ClosedPublic
Actions

Authored by joshuaspence on Jan 21 2015, 7:34 AM.

Tags

None

Referenced Files

	F18803739: D11456.diff
	Sat, Oct 18, 3:24 AM

	F18751003: D11456.id27541.diff
	Sat, Oct 4, 9:31 AM

	F18748460: D11456.id27555.diff
	Fri, Oct 3, 10:45 PM

	F18659162: D11456.id.diff
	Tue, Sep 23, 10:14 AM

	F18658231: D11456.id27555.diff
	Tue, Sep 23, 5:26 AM

	F18637278: D11456.diff
	Sep 17 2025, 6:51 AM

	F18628790: D11456.id27555.diff
	Sep 16 2025, 7:18 AM

	F18627356: D11456.diff
	Sep 16 2025, 3:47 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T6998: After removing `process.exit(1)`, Node exits with error code 0 after throwing

Commits

Restricted Diffusion Commit
rPf61846b46908: Fix Aphlict exit status

Summary

Fixes T6998. Based on https://groups.google.com/d/msg/nodejs/zF7GEoPccqw/n26c6gPaluwJ.

Test Plan

Ran Aphlict and forced an error. Saw error messages in the logs and also saw a non-zero exit status.

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3980
Build 3993: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 27541.Jan 21 2015, 7:34 AM

joshuaspence retitled this revision from to Fix Aphlict exit status.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

joshuaspence added a task: T6998: After removing `process.exit(1)`, Node exits with error code 0 after throwing.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 21 2015, 7:34 AM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Jan 21 2015, 2:44 PM

epriestley edited edge metadata.

epriestley added inline comments.

support/aphlict/server/aphlict_server.js
108–109	Do we need a `return;` here? This does not actually exit the process, does it? Maybe renaming `exit()` to `set_exit_code()` or similar would be more clear.

This revision is now accepted and ready to land.Jan 21 2015, 2:44 PM

(In particular, I would expect --test to cause the server to start.)

joshuaspence added inline comments.Jan 21 2015, 7:40 PM

support/aphlict/server/aphlict_server.js
108–109	Ah yeah, that seems correct.

Rename exit() to set_exit_code()

Fix --test mode

Closed by commit rPf61846b46908: Fix Aphlict exit status (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 21 2015, 7:47 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

support/

aphlict/

server/

aphlict_server.js

10 lines

Diff 27555

support/aphlict/server/aphlict_server.js

Loading...