Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fixes #15 #16

Merged
merged 1 commit into from

3 participants

@KyleAMathews

Image paths where concatenated to the -i flag without a space in between them causing notifications in Linux to fail.

@tj tj merged commit 59919fb into tj:master
@niftylettuce

Can you publish updated version to npm that includes this? The current version 1.5.0 does not have this patch included.

@tj
Owner
tj commented

done :)

ps "Fixes #15" is a terrible commit message haha

@niftylettuce

thx for fixing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 11, 2012
  1. @KyleAMathews

    Fixes #15

    KyleAMathews authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/growl.js
View
2  lib/growl.js
@@ -137,7 +137,7 @@ function growl(msg, options, fn) {
args.push('--' + flag, image)
break;
case 'Linux':
- args.push(cmd.icon + image);
+ args.push(cmd.icon + " " + image);
break;
case 'Windows':
args.push(cmd.icon + '"' + image.replace(/\\/g, "\\\\") + '"');
Something went wrong with that request. Please try again.