dylang / grunt-notify

Automatic Notifications when Grunt tasks fail.
https://npmjs.org/package/grunt-notify
MIT License
921 stars 73 forks source link

Error when trying to use grunt-notify, but no problem when doing it manually #100

Open cyrilgandon opened 9 years ago

cyrilgandon commented 9 years ago

With this config :

module.exports = function(grunt) {
  grunt.initConfig({        
      notify: {
        test: {
          options: {
            message: '"YO"'
          }
        }
      }
    });
    grunt.loadNpmTasks('grunt-notify');
    grunt.registerTask('default', ['notify:test']);
};

I'm not able to see a message. I got the following trace :

C:\temp>grunt -v
Initializing
Command-line options: --verbose

Reading "Gruntfile.js" Gruntfile...OK

Registering Gruntfile tasks.
Initializing config...OK

Registering "grunt-notify" local Npm module tasks.
Reading C:\temp\node_modules\grunt-notify\package.json...OK
Parsing C:\temp\node_modules\grunt-notify\package.json...OK
Loading "notify.js" tasks...OK
+ notify
Loading "notify_hooks.js" tasks...OK
+ notify_hooks
Loading "Gruntfile.js" tasks...OK
+ default

No tasks specified, running default tasks.
Running tasks: default

Running "default" task

Running "notify:test" (notify) task
Verifying property notify.test exists in config...OK
File: [no files]
Options: title="temp", message="\"YO\""
>> [grunt-notify] growl: C:\Utils\growlnotify\growlnotify.COM
>> [growl] cmd: growlnotify
>> [growl] args: /i:"C:/temp/node_modules/grunt-notify/images/grunt-logo.png"  "YO" /t:temp
>> [growl] return_code: 4294967295
Done, without errors.

Note that weird return_code 4294967295. Those both commands trigger the message correctly:

C:\temp>growlnotify /i:"C:/temp/node_modules/grunt-notify/images/grunt-logo.png"  "YO" /t:temp
C:\temp>C:\Utils\growlnotify\growlnotify.COM /i:"C:/temp/node_modules/grunt-notify/images/grunt-logo.png"  "YO" /t:temp

What is going on?

rquadling commented 9 years ago

https://github.com/dylang/grunt-notify/pull/113