Send npmlog to stdout

Looks the default of stderr is causing issue for some CI servers. Fixes #1801
sass · Nov 14, 2016 · e7ad0cb · e7ad0cb
1 parent e0aef95
commit e7ad0cb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/scripts/build.js b/scripts/build.js
@@ -10,6 +10,8 @@ var pkg = require('../package.json'),
   log = require('npmlog'),
   sass = require('../lib/extensions');
 
+log.stream = process.stdout;
+
 /**
  * After build
  *
@@ -27,6 +29,8 @@ function afterBuild(options) {
     'binding.node');
 
   mkdir(path.dirname(install), function(err) {
+    log.stream = process.stderr;
+
     if (err && err.code !== 'EEXIST') {
       log.error('node-sass build', err.message);
       return;
@@ -44,6 +48,7 @@ function afterBuild(options) {
           return;
         }
 
+        log.stream = process.stdout;
         log.info('node-sass build', 'Installed to %s', install);
       });
     });
@@ -128,6 +133,7 @@ function installGitDependencies(options, cb) {
 function build(options) {
   installGitDependencies(options, function(err) {
     if (err) {
+      log.stream = process.stderr;
       log.error('node-sass build', err.message);
       process.exit(1);
     }
@@ -149,7 +155,9 @@ function build(options) {
         return;
       }
 
-      if (errorCode === 127 ) {
+      log.stream = process.stderr;
+
+      if (errorCode === 127) {
         log.error('node-sass build', 'node-gyp not found!');
       } else {
         log.error('node-sass build', 'Build failed with error code: %d', errorCode);

diff --git a/scripts/install.js b/scripts/install.js
@@ -11,6 +11,8 @@ var fs = require('fs'),
   log = require('npmlog'),
   downloadOptions = require('./util/downloadoptions');
 
+log.stream = process.stdout;
+
 /**
  * Download file, if succeeds save, if not delete
  *
@@ -110,6 +112,7 @@ function checkAndDownloadBinary() {
   try {
     mkdir.sync(path.dirname(binaryPath));
   } catch (err) {
+    log.stream = process.stderr;
     log.error('node-sass install', 'Unable to save binary to %s: %s', path.dirname(binaryPath), err);
     return;
   }
@@ -122,6 +125,7 @@ function checkAndDownloadBinary() {
 
   download(sass.getBinaryUrl(), binaryPath, function(err) {
     if (err) {
+      log.stream = process.stderr;
       log.error('node-sass install', err);
       return;
     }