You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Well, it looks like react-scripts build runs node build.js, and when you send SIGTERM to the latter process (as happens with an OOM) it exits silently. Default behavior for node on SIGTERM (aka signal 15) is to exit 15. So this script is probably catching SIGTERM somewhere and exiting "gracefully."
process.on('SIGTERM', function () {
process.exit(15);
});
to the end of react-scripts/scripts/build.js fixes the problem. I'll add a couple more signals, console.log an error message, and submit a pull request -- cool? Is there a cleanup hook I should call?
Activity
gaearon commentedon Feb 7, 2017
Agreed it’s an issue, do you have any idea how to solve this?
brandones commentedon Feb 7, 2017
Well, it looks like
react-scripts buildrunsnode build.js, and when you send SIGTERM to the latter process (as happens with an OOM) it exits silently. Default behavior for node on SIGTERM (aka signal 15) is to exit 15. So this script is probably catching SIGTERM somewhere and exiting "gracefully."brandones commentedon Feb 7, 2017
Adding
to the end of
react-scripts/scripts/build.jsfixes the problem. I'll add a couple more signals, console.log an error message, and submit a pull request -- cool? Is there a cleanup hook I should call?gaearon commentedon Feb 7, 2017
@brandones Sounds great, thanks!
Make build exit with error code when interrupted
Make build exit with error code when interrupted (#1496)
gaearon commentedon Feb 11, 2017
Can we consider this resolved by #1496?
gaearon commentedon Feb 11, 2017
Going to close in hope it’s resolved but please reopen if not.
Make build exit with error code when interrupted (react#1496)
Make build exit with error code when interrupted (react#1496)
Make build exit with error code when interrupted (react#1496)
Make build exit with error code when interrupted (react#1496)
Merge upstream changes from create-react-app 0.9.x (#84)