Display pid in already running message (#3131)

Fixes #3106

Display pid in already running message (#3131)
Fixes #3106
2cff8db0 · Rasmus Eneman · Joe Haddad · 89bf273e · 2cff8db0
Commit 2cff8db0 authored 7 years ago by Rasmus Eneman Committed by Joe Haddad 7 years ago
Hide whitespace changes
Inline Side-by-side

Showing

with 5 additions and 2 deletions
+5 -2
--- a/packages/react-dev-utils/getProcessForPort.js
+++ b/packages/react-dev-utils/getProcessForPort.js
@@ -48,9 +48,11 @@ function getProcessCommand(processId, processDirectory) {
    execOptions
  );
+  command = command.replace(/\n$/, '')
  if (isProcessAReactApp(command)) {
    const packageName = getPackageNameInDirectory(processDirectory);
-    return packageName ? packageName + '\n' : command;
+    return packageName ? packageName : command;
  } else {
    return command;
  }
@@ -68,7 +70,8 @@ function getProcessForPort(port) {
    var processId = getProcessIdOnPort(port);
    var directory = getDirectoryOfProcessById(processId);
    var command = getProcessCommand(processId, directory);
-    return chalk.cyan(command) + chalk.blue('  in ') + chalk.cyan(directory);
+    return chalk.cyan(command) + chalk.grey(' (pid ' + processId + ')\n') + 
+      chalk.blue('  in ') + chalk.cyan(directory);
  } catch (e) {
    return null;
  }