Homec4science

Try `nodejs` before `node` when starting notification server

Authored by epriestley <git@epriestley.com> on Jun 7 2014, 22:56.

Description

Try nodejs before node when starting notification server

Summary: Fixes T5278. This isn't completely perfect (if you have the other node binary, it will fail to detect that it's wrong) but we can maybe wait for that to happen and devise some kind of "is this binary really node?" test if users actually hit it.

Test Plan: Faked things, hit the error; unfaked them and hit the normal flow.

Reviewers: joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T5278

Differential Revision: https://secure.phabricator.com/D9419

Details

Committed
epriestley <git@epriestley.com>Jun 7 2014, 22:56
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbfc594b3eb78: Add project tags to Pholio
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH39ca090d154b: Try `nodejs` before `node` when starting notification server (authored by epriestley <git@epriestley.com>).Jun 7 2014, 22:56