[fix] Add correct header name, fixes #499

nodejitsu · May 20, 2014 · 5266f94 · 5266f94
1 parent 0d3811b
commit 5266f94
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/lib/jitsu.js b/lib/jitsu.js
@@ -372,7 +372,6 @@ jitsu.showError = function (command, err, shallow, skip) {
         }
         else if (~err.result.error.indexOf('npm exited')) {
           jitsu.log.error(err.result.error);
-          jitsu.log.error("npm installation failed, please double check your package.json dependencies");
         } else {
           jitsu.log.error(err.result.error);
         }

diff --git a/lib/jitsu/package.js b/lib/jitsu/package.js
@@ -435,14 +435,14 @@ package.updateTarball = function (version, pkg, existing, firstSnapshot, callbac
           }
           res.pipe(process.stdout);
           res.on('end', function () {
-            if (res.trailers['x-build-failure']) {
-              var error = new Error('Nodejitsu Error: Build Failure, Check your dependencies');
-              error.result = { error: res.trailers['x-npm-failure'].toString() };
+            if (res.trailers && res.trailers['x-build-failure']) {
+              var error = new Error('Nodejitsu Error: Build Failure, please check the build failure above');
+              error.result = { error: res.trailers['x-build-failure'].toString() };
               return callback(error);
             }
             jitsu.log.silly('Done creating snapshot ' + version.magenta);
             callback(null, version, pkg);
-          })
+          });
         });
 
         if (emitter && !jitsu.config.get('raw') && process.stdout.isTTY ) {