Add Notification Center Support on 10.8 #22

Merged
merged 4 commits into from Sep 6, 2012

Projects

None yet

2 participants

@drudge
Contributor
drudge commented Sep 6, 2012

Closes #20. This uses node-ffi, I'm sure there is a better way. What do you think?

@tj
Owner
tj commented Sep 6, 2012

hmm what do we need node-ffi for? the sync exec?

@drudge
Contributor
drudge commented Sep 6, 2012

Yeah, it was a dep on the execSync package.

On Sep 6, 2012, at 2:42 PM, TJ Holowaychuk notifications@github.com wrote:

hmm what do we need node-ffi for? the sync exec?


Reply to this email directly or view it on GitHub.

@tj
Owner
tj commented Sep 6, 2012

howcome we can't just use the async exec()? fire-and-forget style or optional callback like before

@drudge
Contributor
drudge commented Sep 6, 2012

Updated with your which command from nshell to remove the dependency on execSync and node-ffi.

@tj tj merged commit 1b78465 into tj:master Sep 6, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment