Kouhei Sutou
kou****@clear*****
2012年 5月 8日 (火) 14:58:53 JST
須藤です。 In <CABnM****@mail*****> "[groonga-dev,00823] Re: 連続INSERT時に全文検索でエラー" on Tue, 8 May 2012 13:22:12 +0900, "IWAI, Masaharu" <iwaim****@gmail*****> wrote: > 2012年5月8日11:41 Kouhei Sutou <kou****@clear*****>: >> あ、upgradeするときは一度uninstallしてからinstallし直したいん >> です。そうしないと、新しいha_mroonga.soを読みなおしてくれない >> ので。これだとupgrade時はuninstallもinstallもしないですよね。 > > あー、なるほど。だから昨日Twitterであんな感じにおっしゃってたんですね。 > 昨日須藤さんが仰ってた通りでOKですね。 > > https://github.com/iwaim/mroonga/commit/08718960b841ea8afe912648d0dc559029f8db86#diff-0 > # Fedoraの方の「fi」消し忘れは無視してください。 おぉ! >>> こちらですが、上記修正を今やっても「次のリリース」のときは >>> mysql-mroonga-2.01-0のpostunスクリプトが走るので消されちゃいますね……。 >> >> そうですね。。。 >> 今の%postを%posttransにすればうまくいきそうな気がするんです >> が、そうでもないですかねぇ。。。 > > それでも行けそうですけど、トランザクション内の方がよさそうなのと、 > そのあたりはあまり使われないので、全部そっちにもってっちゃうとspec > 読む人がはまる懸念があるので、triggerpostunでバージョン指定いれちゃった > 方がいいんじゃないですかねぇ。 まぁ、何回もアンインストールされてもエラーメッセージがでるだ けで実害はないので、古いバージョンからのアップグレードのとき は気にしなくてもよい気がしてきました! > あと、MySQLのプラグインは詳しくないんですが、postunのやつは、 > ファイルがあるうちに実行するpreunの方が美しい気もしています。 たしかにそうですね! 1. ifのやつと 2. それに合わせてインデントを直したやつと 3. postunをpreunにしたやつを pull requestしてもらえるとうれしいです! (できれば、それぞれを別のコミットにして計3コミットくらいに なっているとうれしい感じがします!が、無理そうなら無理をしな くても大丈夫です!) -- 須藤 功平 <kou****@clear*****> 株式会社クリアコード <http://www.clear-code.com/> (03-6231-7270) groongaサポート: http://groonga.org/ja/support/ プログラミングが好きなソフトウェア開発者を募集中: http://www.clear-code.com/recruitment/