search

mikaelbr / gulp-notify

gulp plugin to send messages based on Vinyl Files or Errors to Mac OS X, Linux or Windows using the node-notifier module. Fallbacks to Growl or simply logging
MIT License
592 stars 40 forks source link

Doesn't work under Tmux #73

Open girvo opened 9 years ago

girvo commented 9 years ago

Hi there,

As per issue #70, gulp-notify doesn't work under tmux correctly. According to that issue, it's because of terminal-notifier not handling it properly, gulp-notify uses I assume? I've updated to the latest gulp-notify, but the issue still exists despite terminal-notifier reporting it as fixed: https://github.com/alloy/terminal-notifier/issues/115

Any ideas?

Edit: Actually, I've found how to fix it -- for those using Tmux on OS X, you'll need to run brew up && brew install reattach-to-user-namespace, then add this line to your ~/.tmux.conf file: 

set-option -g default-command "which reattach-to-user-namespace > /dev/null && reattach-to-user-namespace -l $SHELL || $SHELL"

I found instructions from here: https://github.com/dylang/grunt-notify/issues/92

mikaelbr commented 9 years ago

node-notifier should use v1.6.2 (from node-notifier@4.1.2), so if the tmux issue is fixed it should work as expected, but there might be something messing it up. What version are you using both of gulp-notify and node-notifier?

tanrax commented 9 years ago

+1 Me too node v4.2.1 npm v3.3.9 node-notifier@4.3.1 gulp-notify@2.2.0

I'm on OS X using Tmux through iterm2