Colorful output

[jharding]

Colorful output

[Dremora]

Still an issue in my case since the command I'm trying to execute doesn't have any flags to enable colors.

[dickeylth]

@gwicksted I've created a very simple demo: https://github.com/dickeylth/grunt-exec-issue, you can check out and debug.

[nonlux]

Sorry, color works fine!
Solution

[dickeylth]

+1 with @Dremora

[gwicksted]

@Dremora and @dickeylth can you detail what the command is you are executing and what output is displayed + what you expected for color highlighting?

Also, have you tried this type of configuration (the stdout: 'inherit' is the important part):

module.exports = function(grunt) {
  grunt.initConfig({
    exec: {
      something: {
        command: process.platform === 'win32' ? 'dir' : 'ls -la'
      , stdout: 'inherit'
      , stderr: 'inherit'
      }
    }
  });
};

[gwicksted]

I have reproduced the issue and implemented a fix - it is a major change due to the capabilities of child_process.exec. The next release (2.0.0) will contain this capability.

[gwicksted]

Please try [email protected] aka grunt-exec@next for the new behavior. grunt-exec@latest is still the stable 1.0.1 release. One of the tests is failing so I'm looking into it but things are otherwise stable.

[dickeylth]

@gwicksted I've updated and retried, and unfortunately it still won't work:

Plus my env:

grunt-exec-issue ➤ npm --version 
3.10.9

[email protected] (/usr/local/bin/node)
[email protected] (/usr/local/lib/node_modules/tnpm/node_modules/.2.16.0@npminstall/lib/index.js)
prefix=/usr/local (load prefix from: $ npm config get prefix)
darwin x64 16.3.0

[gwicksted]

Hi @dickeylth , please try with the stdout: 'inherit' option. Example change to your grunt file: https://github.com/gwicksted/grunt-exec-issue/tree/patch-1

[gwicksted]

[email protected] (aka grunt-exec@next) now supports the shortcut option stdio: 'inherit'

[dickeylth]

@gwicksted It worked now, thank you!

[dickeylth]

@gwicksted I'm wondering when would the grunt-exec@next be released?

[gwicksted]

@dickeylth I was holding back the release a bit because it replaces the entire back-end of grunt-exec. That said, I've been using this version since Jan with no issues so I think it's safe to mark it stable.