process: move deprecation warning setup for --debug* args

nodejs · Mar 17, 2019 · f442b54 · f442b54
1 parent 00e7dd2
commit f442b54
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
@@ -219,20 +219,6 @@ Object.defineProperty(process, 'argv0', {
 });
 process.argv[0] = process.execPath;
 
-// Handle `--debug*` deprecation and invalidation.
-if (process._invalidDebug) {
-  process.emitWarning(
-    '`node --debug` and `node --debug-brk` are invalid. ' +
-      'Please use `node --inspect` or `node --inspect-brk` instead.',
-    'DeprecationWarning', 'DEP0062', undefined, true);
-  process.exit(9);
-} else if (process._deprecatedDebugBrk) {
-  process.emitWarning(
-    '`node --inspect --debug-brk` is deprecated. ' +
-      'Please use `node --inspect-brk` instead.',
-    'DeprecationWarning', 'DEP0062', undefined, true);
-}
-
 // TODO(jasnell): The following have been globals since around 2012.
 // That's just silly. The underlying perfctr support has been removed
 // so these are now deprecated non-ops that can be removed after one

diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
@@ -18,6 +18,20 @@ function prepareMainThreadExecution() {
       setupCoverageHooks(process.env.NODE_V8_COVERAGE);
   }
 
+  // Handle `--debug*` deprecation and invalidation.
+  if (process._invalidDebug) {
+    process.emitWarning(
+      '`node --debug` and `node --debug-brk` are invalid. ' +
+        'Please use `node --inspect` or `node --inspect-brk` instead.',
+      'DeprecationWarning', 'DEP0062', undefined, true);
+    process.exit(9);
+  } else if (process._deprecatedDebugBrk) {
+    process.emitWarning(
+      '`node --inspect --debug-brk` is deprecated. ' +
+        'Please use `node --inspect-brk` instead.',
+      'DeprecationWarning', 'DEP0062', undefined, true);
+  }
+
   // Only main thread receives signals.
   setupSignalHandlers();