Notify compile error with Growl(GNTP). #19

Closed
ponkore opened this Issue Aug 4, 2012 · 6 comments

Projects

None yet

2 participants

Contributor
ponkore commented Aug 4, 2012

[Enhancement]
misaki.server でテンプレートをガシガシいじっているときに、ミスをしてコンパイルエラーになってしまうことがままありますが、その時 Growl 等で通知を出していただけるとありがたいのですが...(ブラウザとEmacsが画面を占領しているので、エラーを起こした時に何度ブラウザを reload しても変更が反映されない、ということがよくあるので...)。

https://gist.github.com/3254119

ちなみに私が使っている Growl 1.4 (AppStore 経由でインストール) ではどうも UDP では繋がらないようで、 clj-growl は使えませんでした。

(あと、上記 gist で使っている clj-gntp の中身はちょっと微妙な感じ(引数の渡し方とか。Javaから呼び出すことを意識している?)なので、独自に再実装したほうがよいのかもしれません。あと、上記 gist は私の Mac でしかテストしていません...)

Owner
liquidz commented Aug 4, 2012

#17 が完了したら対応します!

ちなみにGrowlでUDPが繋がらない件ですが
設定の「ネットワーク > 受信される通知を聞く、リモートアプリケーション登録を許可」に
チェックを入れても繋がらない状況でしょうか?

Contributor
ponkore commented Aug 4, 2012

私のところのダイアログ Growl v1.4
ネットワーク > 受信される通知を聞く、というのがありません...。

対応は例によって全く急ぎませんので、気が向いた時にでもふらっとしていただければありがたいです。
Growl で UDP がつながらない件ですが、公式ドキュメントには特に記述は見当たらなかったのですが、
ruby-growl ライブラリのドキュメント(のnew の説明のところ)に以下のような記述がありました。

‘GNTP’

Use GNTP connections. GNTP is supported by Growl 1.3 and newer and by Growl for Windows.

‘UDP’

Uses the UDP growl protocol. UDP growl is supported by Growl 1.2 and older.

なので、いっそのこと GNTP オンリーにしておいたほうが良いのかもしれません。

Contributor
ponkore commented Aug 4, 2012

よく考えたら 1.2 以前もニーズがありそうですね(MacPorts で入るのは 1.1.6 ですし...)。
GNTPオンリーは言いすぎでした、すみません。
ネット上で調べて見る限りでは、UDP版実装のほうが多いような感じです。
(emacs lisp 版 https://github.com/elim/emacs-growl/blob/master/growl.el なんてのもあるし...)

@liquidz liquidz was assigned Aug 27, 2012
@liquidz liquidz added a commit that referenced this issue Sep 5, 2012
@liquidz liquidz #19 add misaki.util.notify 2f5fe5a
@liquidz liquidz added a commit that referenced this issue Sep 8, 2012
@liquidz liquidz #19 update notify title and message 9a0aacc
Owner
liquidz commented Sep 8, 2012

devブランチで対応しましたのでご確認いただけますか?
デフォルトでは通知オフなので _config.clj 内で以下のように指定してください。

{
 :notify? true
 }

なおgrowlとのgntp, udp通信についてはmisakiのコアとは別になるので
ひとまず対応優先でclj-gntp, clj-growlを使っています。

Contributor
ponkore commented Sep 8, 2012

devブランチ確認させていただきました。OKです!!
しかもエラーから回復したら FIXED になるおまけ付き!!!すばらしいです。

ありがとうございます。

Owner
liquidz commented Sep 8, 2012

ご確認ありがとうございます!
masterにマージしましたのでクローズします。

@liquidz liquidz closed this Sep 8, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment