WIP #217 added failure message on failed 'webgme start'

webgme · Nov 9, 2017 · 7384aea · 7384aea
1 parent 1e59a96
commit 7384aea
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/bin/webgme-start b/bin/webgme-start
@@ -12,5 +12,9 @@ program
     .parse(process.argv);
 
 if (program.args.length === 0) {  // action will not be called
-    manager.start(program, nop);
+    manager.start(program, err => {
+        if (err) {
+            process.exit(1);
+        }
+    });
 }
diff --git a/src/BaseManager.js b/src/BaseManager.js
@@ -44,12 +44,14 @@ BaseManager.prototype._start = function (root, callback) {
         this._logger.write('Installing dependencies...');
         npm.install(err => {
             if (err) {
+                this._logger.error(`Could not install dependencies: ${err.message}`);
                 return callback(err);
             }
             if (!exists(webgmePath)) {
                 this._logger.write('Installing webgme...');
                 npm.install('webgme', err => {
                     if (err) {
+                        this._logger.error(`Could not install webgme dependency: ${err.message}`);
                         return callback(err);
                     }
                     npm.start();