External process changes

VernonGrant · Nov 13, 2016 · 7ff5d27 · 7ff5d27
1 parent 3297aee
commit 7ff5d27
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/lib/main.js b/lib/main.js
@@ -38,14 +38,17 @@ let breakPackage = {
                 breakServerProcess.startServer().on('message', function(m) {
                     socket.open();
                 }).on('error', function(err){
-                    atom.notifications.addWarning(
-                        'Break: Node not found.',
+
+                    // alert user if an error is recieved
+                    atom.notifications.addError(
+                        'Break;',
                         {
-                            description: 'Break; requires that you have **Nodejs** installed on your system and available in your **path**. Node can be downloaded from the following link **[nodejs.org](https://nodejs.org/en/)**',
-                            detail: 'Please install Nodejs before using Break;',
+                            description: 'If this error continues please help us by opening an issue **[Here](https://github.com/VernonGrant/break/issues)**',
+                            detail: 'Failed to start child process.',
                             dismissable: true
                         }
                     );
+
                 });
 
             }

diff --git a/lib/serverProcess.js b/lib/serverProcess.js
@@ -12,19 +12,15 @@ export default class ServerProcess{
     constructor(port, host = hostname){
         this.port = port;
         this.host = host;
-        this.childProcess = null;
     }
 
     /**
      * Starts the external server process.
-     *
-     * @returns {ChildProcess}
      */
-    startServer(){
-        this.childProcess = childProcess.spawn('node', [serverScriptPath], {
+    startServer(socket){
+        return childProcess.fork(serverScriptPath, [], {
             stdio: ['inherit', 'inherit', 'inherit', 'ipc']
         });
-        return this.childProcess;
     }
 
     /**