cre-ne-jp / rgrb

irc.cre.jp 系 IRC サーバ群で用いられている汎用チャットボット
http://www.cre.ne.jp/services/irc/bots/rgrb
MIT License
4 stars 0 forks source link

Bcdice: BCDice のバージョンを出力できるようにする #157

Closed koi-chan closed 5 years ago

koi-chan commented 5 years ago

https://github.com/bcdice/BCDice/blob/master/src/configBcDice.rb#L8

本家の BCDice の configBcDice.rb 内で、バージョンを定義する変数が更新されるようになった。 .bcdice-version コマンドの応答に、コミット ID だけでなく、数字のバージョンを追加する。

koi-chan commented 5 years ago

当該ファイルを require すると、大量のグローバル変数が有効になり、バグを生む危険がありそうです。 テキストファイルとして解析して、バージョン番号だけ抜き出したほうがよさそうに思えるのですが、いかがでしょうか。

koi-chan commented 5 years ago

cgiDiceBot.rb -> bcdiceCore.rb -> configBcdice.rb というつながりで、すでに require され、グローバル変数が有効になっていました。

ochaochaocha3 commented 5 years ago

テキストファイルとして解析することに賛成です。BCDiceはv3にならないとグローバル変数を使わないように大改造されないと思うので、今のところはその方針でいきましょう。

ochaochaocha3 commented 5 years ago

会話の流れを追えてなくて申し訳ございませんでした。グローバル変数が有効になっていたということは、$bcDiceVersion を直接見ればよかったです。