機能 #610
完了
rnotify.rb 内でモジュール名が間違っている?
0%
説明
NetBSD方面のパッケージシステム pkgsrc の mikutter において
http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/net/mikutter/patches/patch-core_plugin_libnotify_rnotify.rb?rev=1.1&content-type=text/x-cvsweb-markup
という Gtk ⇒ Gdk なモジュール名の修正パッチが当てられているのですが、
これは正しい修正でしょうか?
pkgsrcにこれをコミットされた方からRedmine契約の申請が出ているんじゃないか
という気もするのですが、pkgsrcに限った話でもないと思うので
先走ってチケット登録してしまいます。
関連するチケット
toshi_a 初音 さんが10年以上前に更新
- ステータス を 新規 から 実装待ち に変更
結論から言えば、rnotifyは事実上使用されていないため、発見されていませんでした。その修正自体は問題ないのではないでしょうか。
簡単に経緯を説明すると、この変更を取り込んだあと、rnotify自体の更新が追いついておらず、Ubuntuの当時の最新版やArchで、通知時に高確率でクラッシュすることがあるということがわかりました。それから、パッチを書いた人に相談しましたが話が止まっており、多くの環境ではnotify-sendコマンドを毎回起動しても十分なパフォーマンスが発揮できるため、事実上ほとんどしようされていないようだ、という状況です。
libnotifyプラグインは、gemがあるときと無いときで動作が変わるプラグインです。rnotify が使用できる環境ならrnotifyを使って通知を送り、なければnotify-sendコマンドを使う、という動作です。
現在のmikutterの考え方からすると、このような挙動は好ましくなく、どちらか片方を残し、別のプラグインを用意し、ユーザやパッケージメンテナがプラグインを選択するべきです。
このチケットでそのことを思い出したので、0.3では、現在の環境での検証の結果によって、どちらかのコードを削除しようと思いますが、いかがでしょうか。
Izumi Tsutsui さんが10年以上前に更新
このチケットでそのことを思い出したので、0.3では、現在の環境での検証の結果によって、
どちらかのコードを削除しようと思いますが、いかがでしょうか。
問題ないと思います。
pkgsrcでは pkgsrc/sysutils/ruby-notify というのが存在するので
それをインストールしている環境では問題が発現したということではないかと思います。
ちなみに pkgsrc/net/mikutter 的には rnotify には依存するようには
なっていません。
匿名ユーザー さんが10年以上前に更新
契約が滞りすぎて、報告しようとしていたのをすっかり忘れていましたが、
Izumi Tsutsui は書きました:
pkgsrcでは pkgsrc/sysutils/ruby-notify というのが存在するので
それをインストールしている環境では問題が発現したということではないかと思います。
pkgsrc 的には、rnotify は sysutils/ruby-libnotify の方です。
こちらは mikutter 以前に、単体でテストを行ってもまともに動作しないような状態です(ハングアップする)。
sysutils/ruby-notify で入る notify gem のほうは、notfy-send コマンドとか、 libnotify とか、growl とか、その他諸々の中から利用する通知方法を選択できるラッパーのようなモジュールです。
toshi_a 初音 さんがほぼ6年前に更新
- 次のチケットと重複 バグ #1237: rnotify.rbのGtk::WebImageLoader.local_path(user[:profile_image_url])はたぶんエラーになる？ を追加