process: move deprecation warning setup for --debug* args · nodejs/node@cf1117a (original) (raw)

2 files changed

lines changed

Original file line number Diff line number Diff line change
@@ -219,20 +219,6 @@ Object.defineProperty(process, 'argv0', {
219 219 });
220 220 process.argv[0] = process.execPath;
221 221
222 -// Handle `--debug*` deprecation and invalidation.
223 -if (process._invalidDebug) {
224 -process.emitWarning(
225 -'`node --debug` and `node --debug-brk` are invalid. ' +
226 -'Please use `node --inspect` or `node --inspect-brk` instead.',
227 -'DeprecationWarning', 'DEP0062', undefined, true);
228 -process.exit(9);
229 -} else if (process._deprecatedDebugBrk) {
230 -process.emitWarning(
231 -'`node --inspect --debug-brk` is deprecated. ' +
232 -'Please use `node --inspect-brk` instead.',
233 -'DeprecationWarning', 'DEP0062', undefined, true);
234 -}
235 -
236 222 // TODO(jasnell): The following have been globals since around 2012.
237 223 // That's just silly. The underlying perfctr support has been removed
238 224 // so these are now deprecated non-ops that can be removed after one
Original file line number Diff line number Diff line change
@@ -18,6 +18,20 @@ function prepareMainThreadExecution() {
18 18 setupCoverageHooks(process.env.NODE_V8_COVERAGE);
19 19 }
20 20
21 +// Handle `--debug*` deprecation and invalidation.
22 +if (process._invalidDebug) {
23 +process.emitWarning(
24 +'`node --debug` and `node --debug-brk` are invalid. ' +
25 +'Please use `node --inspect` or `node --inspect-brk` instead.',
26 +'DeprecationWarning', 'DEP0062', undefined, true);
27 +process.exit(9);
28 +} else if (process._deprecatedDebugBrk) {
29 +process.emitWarning(
30 +'`node --inspect --debug-brk` is deprecated. ' +
31 +'Please use `node --inspect-brk` instead.',
32 +'DeprecationWarning', 'DEP0062', undefined, true);
33 +}
34 +
21 35 // Only main thread receives signals.
22 36 setupSignalHandlers();
23 37