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

	Unknown Object (File)
	Sun, Apr 14, 9:48 PM

	Unknown Object (File)
	Sun, Apr 14, 9:48 PM

	Unknown Object (File)
	Sun, Apr 14, 9:48 PM

	Unknown Object (File)
	Sun, Apr 14, 9:47 PM

	Unknown Object (File)
	Sun, Apr 14, 9:42 PM

	Unknown Object (File)
	Sun, Apr 14, 8:57 PM

	Unknown Object (File)
	Thu, Apr 11, 10:05 AM

	Unknown Object (File)
	Mon, Apr 8, 4:51 PM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 27556

support/aphlict/server/aphlict_server.js

Loading...